CodeSonar : Pvm : Null Pointer Dereference

Pvm : Pvm analysis 1 : Null Pointer Dereference at cons.c:681

Categories:	LANG.MEM.NPD CWE:476
Warning ID:	2905.2975
Procedure:	checkmsgs
Trace:	View
Modified:	Wed Sep 2 12:52:26 2009 show details hide details

Priority:	None
State:	None
Finding:	None
Owner:	None
	edit properties

Legend	[ X ]
Warning Location Contributes Parse Error Other Warning Two or More Loop Iterations On Execution Path Comment Macro Preprocessor Include Keyword Preprocessed Away

Legend
Warning Location	Contributes	Parse Error
Other Warning	Two or More Loop Iterations	On Execution Path
Comment	Macro	Preprocessor
Include	Keyword	Preprocessed Away

Source | Language: C

Hide Legend

Problem	Line	Source

			/u1/paul/SATE/2010/c/pvm/pvm3/console/cons.c

			Enter checkmsgs

{

int cc;

int len, cod, src;

int n;

struct job *jp;

int *dtids;

int i, j;

int more, tmp;

struct job *jpnext;

/* Shit, really need context here to keep old pvm_nrecv(-1,-1)... */

666

/* Should use 2 contexts, one for original console msgs and one */

667

/* for tracer stuff... */

668

669

/* Get Host Add Notifies */

670

while ((cc = pvm_nrecv(-1, HostsAddedTag )) > 0) {

671

pvm_upkint(&n, 1, 1);

true

672

dtids = TALLOC(n, int, ""); /* Uninitialized Variable (ID: 2909.2981) */

673

pvm_upkint(dtids, n, 1);

674

printf("\nConsole: %d new host%s added\n", n, (n == 1 ? "" : "s"));

675

pvm_freebuf(cc );

676

if (!pvm_config(&nhosts, &narchs, &hostlist )) {

677

fputs(" HOST DTID ARCH SPEED\n",

678

stdout);

679

for (j = n; j-- > 0; )

680

for (i = nhosts; i-- > 0; ) {

dtids <= 4095

681

if (dtids[j] == hostlist[i].hi_tid) { /* Null Pointer Dereference */

682

printf("%24s %8x %8s%8d\n",

hostlist[i].hi_name,

hostlist[i].hi_tid,

hostlist[i].hi_arch,

hostlist[i].hi_speed);

687

break;

Preconditions

&$unknown_3216061 >= 1
&$unknown_3216064 >= 2
&$unknown_3216068 = 0
&$unknown_3216069 >= 1
&$unknown_3216070 = 1
pvmtoplvl != 0

Postconditions

cc' = &$unknown_3216061
dtids' = &$unknown_3216068
i' = &$unknown_3216069 - 1
j' = 0
n' = &$unknown_3216064
nhosts' = &$unknown_3216069

Change Warning 2905.2975 : Null Pointer Dereference

Priority:
State:
Finding:
Owner:
Note: