Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Null Pointer Dereference  at ddpro.c:3137

Categories: LANG.MEM.NPD CWE:476
Warning ID: 2589.2623
Similar Warnings: 2589.2621
Procedure: dm_db
Trace: View
Modified: Wed Sep 2 12:41:54 2009   show 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     
ProblemLineSource
   /u1/paul/SATE/2010/c/pvm/pvm3/src/ddpro.c
   Enter dm_db
 2943 dm_db(hp, mp) 
 2944         struct hostd *hp;
 2945         struct pmsg *mp;
 2946 {
 2947         int opcode;                                     /* op requested */ 
 2948         int tid;
 2949         int req;                                        /* index requested */ 
 2950         int flags;
 2951         char *name = 0;                         /* class name */ 
 2952         struct pmsg *mp2 = 0;           /* reply */ 
 2953         struct pmsg *mp3 = 0;           /* data message */ 
 2954  
 2955         struct waitc *wp, *wp2;         /* wait ctx ptrs (notify, recvinfo */ 
 2956         struct pmsg *mp4 = 0;           /* notify forward message */ 
 2957         struct hostd *hp2;                      /* remote notify host */ 
 2958  
 2959         struct pvmmclass *np, *np2;     /* reset pointers */ 
 2960         struct pvmmentry *ep, *ep2;     /* reset pointers */ 
 2961         int *noresets;                          /* noreset tids */ 
 2962         int nnr;                                        /* # of noreset tasks */ 
 2963         int found;
 2964         int cc;
 2965         int i;
 2966         int notified;
 2967  
 2968         hp = hp;
 2969  
 2970         if (upkint(mp, &opcode) || upkint(mp, &tid) 
 2971         || upkstralloc(mp, &name) || upkint(mp, &req) || upkint(mp, &flags))
 2972                 goto badformat;
 2973  
 2974         mp2 = mesg_new(0);
 2975         mp2->m_dst = mp->m_src;   /* Null Pointer Dereference (ID: 2599.2633) */
 2976         mp2->m_tag = DM_DBACK;
 2977         mp2->m_wid = mp->m_wid;
 2978  
 2979         switch (opcode) {
 2980  
 2981         case TMDB_PUT: 
 2982                 mp3 = mesg_new(0);
 2983                 if (pmsg_unpack(mp, mp3))
 2984                         goto badformat;
 2985                 if ((req = mb_insert(tid, name, req, flags, mp3)) < 0)
 2986                         pmsg_unref(mp3);
 2987                 else {
 2988  
 2989                         /* check for any pending requests for this mbox entry */ 
   ...
 3070                                 pmsg_unref(mp3);
 3071                         }
 3072                 }
 3073                 break;
 3074  
 3075         case TMDB_NAMES: 
 3076                 pkint(mp2, 0);
 3077                 req = mb_names(tid, name, mp2);
 3078                 break;
 3079  
 3080         case TMDB_RESET: 
 3081                 if ( upkint(mp, &nnr) )
 3082                         goto badformat;
 3083                 noresets = TALLOC( nnr, int, "int" );
 3084                 for ( i=0 ; i < nnr ; i++ ) {
 3085                         if ( upkint(mp, &(noresets[i])) ) {
 3086                                 PVM_FREE(noresets);   /* Free Null Pointer (ID: 2592.2627) */
 3087                                 goto badformat;
 3088                         }
 3089                 }
 3090                 pkint(mp2, 0);
 3091                 for (np = pvmmboxclasses->mc_link; np != pvmmboxclasses;
 3092                                 np = np2) {
 3093                         np2 = np->mc_link;
 3094  
 3095                         /* If name passed in, only wipe mboxes in that class */ 
 3096                         if ( *name == '\0' || !strcmp( np->mc_name, name ) ) {   /* Null Pointer Dereference (ID: 2594.2628) */
 3097  
 3098                         for (ep = np->mc_ent->me_link; ep != np->mc_ent; ep = ep2) {
 3099                                 ep2 = ep->me_link;
 3100  
 3101                                 /* If index passed in, only wipe that mbox */ 
 3102                                 /* -1 == All Entries */ 
 3103                                 if ( req < 0 || req == ep->me_ind ) {
 3104  
 3105                                 /* Check for Persistency (that thorn in my "side" :-) */ 
 3106                                 if ( ep->me_flags & PvmMboxPersistent ) {
 3107  
 3108                                         /* Task Already Gone?  Spank It. */ 
 3109                                         if (!(ep->me_tid)) {
 3110                                                 me_free(np, ep);
 3111                                                 if (np2->mc_rlink != np)
 3112                                                         break;
 3113                                         }
 3114                                  
 3115                                         /* Check for No-Reset Task */ 
 3116                                         /* (Only if actually killing tasks...) */ 
 3117                                         else if ( flags ) {  /* killtasks */ 
 3118                                                 for ( i=0, found=0 ; i < nnr && !found ; i++ )
 3119                                                         if ( noresets[i] == ep->me_tid )
 3120                                                                 found++;
 3121  
 3122                                                 /* Not a No-Reset, It WILL die soon. */ 
 3123                                                 /* Wait for cleanup. */ 
 3124                                                 if ( !found ) {
 3125                                                         wp = wait_new(WT_RESET);
 3126                                                         wp->wa_on = ep->me_tid;
 3127                                                         wp->wa_tid = pvmmytid;
 3128                                                         wp->wa_dep = 0;
 3129                                                         wp->wa_mesg = (struct pmsg *) NULL;
 3130  
 3131                                                         /* pass on to non-master host */ 
 3132                                                         hp2 = tidtohost(hosts, ep->me_tid);
 3133                                                         if ( hp2 && hp2->hd_hostpart != myhostpart )
 3134                                                         {
true3135                                                                 mp4 = mesg_new(0);
 3136                                                                 pkint(mp4, ep->me_tid);
mp4 <= 40953137                                                                 mp4->m_dst = hp2->hd_hostpart | TIDPVMD;     /* Null Pointer Dereference */
 3138                                                                 mp4->m_tag = DM_RESET;
 3139                                                                 mp4->m_wid = wp->wa_wid;
 3140                                                                 sendmessage(mp4);
Preconditions 
&$unknown_726669 >= 1
&$unknown_726674 = &$unknown_726673 + 136
&$unknown_726674 >= 4232
&$unknown_726684 != 0
&$unknown_726687 = 0
((char*)((char*)*$unknown_726677)[24])[16] >= 0
((char*)((char*)*$unknown_726677)[24])[20] != 0
((char*)$unknown_726677)[24] != ((char*)*$unknown_726677)[24]
hosts->ht_last >= 1
waitlist->wa_wid >= widbase + 2
pvmmboxclasses->mc_link != pvmmboxclasses
$unknown_726677 != pvmmboxclasses
((char*)&$unknown_726686)[4] != myhostpart
$heap_39630 = 0
strlen(&$heap_39630) = 0
numpmsgs = 0
Postconditions 
waitlist->wa_rlink' = &$heap_39632
((char*)$unknown_726673)[8]' = &$unknown_726674
$unknown_726674' = &freepmsgs.m_link
((char*)&$unknown_726674)[8]' = ((char*)&$unknown_726673)[8]
$unknown_726685' = &$heap_39632
ep' = ((char*)*$unknown_726677)[24]
ep2' = *((char*)*$unknown_726677)[24]
flags' = &$unknown_726684
found' = 0
freepmsgs.m_link' = &freepmsgs.m_link
freepmsgs.m_rlink' = &freepmsgs.m_link
bytes_after(&$heap_39630)' = &$unknown_726669
$heap_39630' is allocated by malloc
bytes_before(&$heap_39630)' = 0
bytes_after(&$heap_39631)' = 0
$heap_39631' is allocated by malloc
$heap_39631' is allocated
bytes_before(&$heap_39631)' = 0
$heap_39632' = waitlist
bytes_after(&$heap_39632)' = 80
$heap_39632' is allocated by malloc
$heap_39632' is allocated
bytes_before(&$heap_39632)' = 0
((char*)&$heap_39632)[16]' = widbase + 1
((char*)&$heap_39632)[20]' = 17
((char*)&$heap_39632)[24]' = ((char*)((char*)*$unknown_726677)[24])[20]
((char*)&$heap_39632)[28]' = pvmmytid
((char*)&$heap_39632)[32]' = 0
((char*)&$heap_39632)[56]' = 0
((char*)&$heap_39632)[64]' = 0
((char*)&$heap_39632)[72]' = 0
((char*)&$heap_39632)[8]' = &$unknown_726685
hp' = $param_1
hp2' = &$unknown_726686
i' = 0
lastwid' = 1
mp' = $param_2
mp2' = &$unknown_726673
mp3' = 0
mp4' = 0
name' = &$heap_39630
nnr' = 0
noresets' = &$heap_39631
np' = $unknown_726677
np2' = *$unknown_726677
numpmsgs' = 0
opcode' = 5
req' = ((char*)((char*)*$unknown_726677)[24])[16]
wp' = &$heap_39632



Change Warning 2589.2623 : Null Pointer Dereference

Because they are very similar, this warning shares annotations with warning 2589.2621.
CodeSonar has selected warning 2589.2621 to represent this group of similar warnings. In order to edit this group, you must edit warning 2589.2621.