CodeSonar : Pvm : Null Pointer Dereference

Pvm : Pvm analysis 1 : Null Pointer Dereference at pvmcruft.c:694

Categories:	LANG.MEM.NPD CWE:476
Warning ID:	2539.2570
Procedure:	hex_inadport
Trace:	View
Modified:	Wed Sep 2 12:40:45 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/src/ddpro.c

			Enter dm_htupd

	1744		dm_htupd(hp, mp)
	1745		struct hostd *hp;
	1746		struct pmsg *mp;
	1747		{
	1748		int count; /* number of hosts in message */
	1749		int hh;
	1750		char buf[16]; /* for converting sockaddr */
	1751		struct pmsg *mp2;
	1752
	1753		/* unpack new host table params */
	1754
	1755		newhosts = ht_new (1);
	1756		newhosts->ht_local = hosts->ht_local;
	1757		upkint(mp, &newhosts->ht_serial );
	1758		upkint(mp, &newhosts->ht_master );
	1759		upkint(mp, &newhosts->ht_cons );
	1760
	1761		/* add current hosts to the table */
	1762
	1763		ht_merge(newhosts, hosts );
	1764
	1765		/* unpack new hosts and add to table */
	1766
	1767		/* XXX if we find a host already in table we should kill it with
	1768		XXX hostfail and put the new one in its place */
	1769
	1770		upkint(mp, &count );
	1771		while (count-- > 0) {
	1772		upkint(mp, &hh );

true

1773

hp = hd_new (hh );

	1774		upkstralloc(mp, &hp->hd_name );
	1775		upkstralloc(mp, &hp->hd_arch );
	1776		upkstr(mp, buf, sizeof(buf));

hp <= 4007

1777

hex_inadport(buf, &hp->hd_sad );

					/u1/paul/SATE/2010/c/pvm/pvm3/src/pvmcruft.c

					Enter dm_htupd / hex_inadport

hex_inadport(s, sad)

char *s;

struct sockaddr_in *sad;

693

{

sad <= 4095

694

sad->sin_addr.s_addr = htonl ((unsigned)pvmxtoi(s )); /* Null Pointer Dereference */

					Exit dm_htupd / hex_inadport

Preconditions

&$unknown_455230 >= 1
hosts->ht_hosts[1] >= 0
hosts->ht_last >= 2

Postconditions

__x' = 0
count' = &$unknown_455230 - 1
bytes_after(&$heap_31473)' = 40
$heap_31473' is allocated by malloc
bytes_before(&$heap_31473)' = 0
strlen(&$heap_31473)' = 0
((char*)&$heap_31473)[20]' = hosts->ht_local
hp' = 0
mp' = $param_2
newhosts' = &$heap_31473
s' = &buf[0]
sad' = 88

Change Warning 2539.2570 : Null Pointer Dereference

Priority:
State:
Finding:
Owner:
Note: