Opened 14 years ago

Last modified 14 years ago

#3 new defect

diffusion1d buf variable — at Version 1

Reported by: siegel Owned by:
Priority: major Milestone:
Component: diffusion1d Version:
Keywords: buf Cc:

Description (last modified by siegel)

In diffusion1d_par.c, there is a global variable buf that is a pointer. It is allocated twice (i.e., two mallocs) on rank 0--hence the first is a memory leak.

Also several functions have a local variable named but which shadows the global one. This is probably a bad idea. It would be better to use different names for variables that visible in the same scope.

Change History (1)

comment:1 by siegel, 14 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.