Text  |   XML   |   Visible Warnings:

pvm3.4.6 : pvm3.4.6 analysis 2 : Null Pointer Dereference  at lpvm.c:2564

Categories: LANG.MEM.NPD CWE:476
Warning ID: 22101.29023
Procedure: mroute
Trace: view
Modified: Thu Nov 26 11:29:46 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
   /kat0/fletcher/SATE/2010/pvm3/src/lpvm.c
   Enter mroute
 2365 mroute(mid, dtid, tag, tmout) 
 2366         int mid;                                /* message (or 0) */ 
 2367         int dtid;                               /* dest */ 
 2368         int tag;                                /* type tag */ 
 2369         struct timeval *tmout;  /* get at least one message */ 
 2370 {
 2371         static int alreadyhere = 0;     /* have i called myself? */ 
 2372  
 2373         struct ttpcb *pcbp;                     /* pcb for foreign task */ 
 2374         int s;                                          /* socket */ 
 2375         int sbf;                                        /* temp TC_CONREQ message */ 
   ...
 2378         int l;
 2379 #ifdef SOCKLENISUINT 
 2380 #if defined(IMA_AIX4SP2) || defined(IMA_AIX5SP2)
 2381         unsigned int oslen;
 2382 #else 
 2383         size_t oslen;
 2384 #endif 
 2385 #else 
 2386         int oslen;
 2387 #endif 
 2388         int cc = 0;
 2389         int gotem = 0;                          /* count of received messages */ 
 2390         static struct timeval ztv = { 0, 0 };
 2391         char *addr = 0;
 2392 #ifndef NOUNIXDOM 
 2393         struct sockaddr_un uns;
 2394         char spath[PVMTMPNAMLEN];
 2395 #endif 
 2396  
 2397         if (up = midtobuf(mid)) {
 2398                 up->m_dst = dtid;
 2399                 up->m_tag = tag;
 2400         }
 2401  
 2402         if (alreadyhere) {
 2403                 if (up) {
 2404                         up->m_ref++;
 2405                         txlist[txwp] = up;
 2406                         if (++txwp >= sizeof(txlist)/sizeof(txlist[0]))
 2407                                 txwp = 0;
 2408                         if (txwp == txrp)
 2409                                 abort();
 2410                 }
 2411  
 2412                 /* we can't wait around if reentered */ 
 2413  
 2414                 if (!tmout || tmout->tv_sec || tmout->tv_usec)
 2415                         return PvmNotImpl;
 2416  
 2417                 return 0;
 2418         }
 2419  
 2420         alreadyhere = 1;
 2421  
 2422         /* if sending a message, decide whether we need to route it. */ 
 2423  
 2424         /* Direct Routing is being set up in this section of code   
 2425      * For workstations (not MPP, SHMEM)  
 2426          *      either Unix domain or TCP sockets are used.
 2427          *  task exit notifies are set up for route closure 
 2428          *
 2429          * For MPPs and SHMEM,
 2430          *      connections to local tasks are always direct and require no 
 2431          *      handshake negotiation.  
 2432         */ 
 2433  
 2434         if (up) {        /*  We've got something to send */ 
 2435  
 2436 #if !defined(IMA_MPP) /* This is the workstation direct route setup */ 
 2437  
 2438                 /* Checks made (Workstation):
 2439                  *      o route option is direct 
 2440                  *      o destination is given 
 2441                  *      o destination is a task 
 2442                  *  o destination is not me 
 2443                  *  o could not find task process control block         
 2444                 */ 
 2445  
 2446                 if (pvmrouteopt == PvmRouteDirect 
 2447                         && dtid 
 2448                         && TIDISTASK(dtid) 
 2449                         && dtid != pvmmytid 
 2450                         && !ttpcb_find(dtid))
 2451                 {
 2452  
 2453 #if !defined(NOUNIXDOM)
 2454                         /* Create a UNIX domain socket to build the direct route */ 
 2455  
 2456                         if ((pvmmytid & TIDHOST) == (dtid & TIDHOST)) {
 2457                                 if ((s = socket(AF_UNIX, SOCK_STREAM, 0)) == -1) {
 2458                                         pvmlogperror("mroute() socket");
 2459  
 2460                                 } else {
 2461                                         BZERO((char*)&uns, sizeof(uns));
 2462                                         uns.sun_family = AF_UNIX;
 2463                                         spath[0] = 0;
 2464                                         (void)PVMTMPNAMFUN(spath);
 2465                                         strcpy(uns.sun_path, spath);   /* Buffer Overrun (ID: 22105.29027) */
 2466  
 2467                                         oslen = sizeof(uns);
 2468                                         if (bind(s, (struct sockaddr*)&uns, oslen) == -1) {
 2469                                                 pvmlogperror("mroute() bind");
 2470                                                 (void)close(s);
 2471                                                 s = -1;
 2472  
 2473                                         } else {
 2474                                                 pcbp = ttpcb_creat(dtid);
 2475                                                 pcbp->tt_fd = s;   /* Null Pointer Dereference (ID: 22104.29026) */
 2476                                                 addr = pcbp->tt_spath = STRALLOC(spath);   /* Null Pointer Dereference (ID: 22103.29025) */
 2477                                                 pcbp->tt_state = TTCONWAIT;
 2478                                         }
 2479                                 }
 2480  
 2481                         } else {
 2482 #endif /* !NOUNIXDOM */ 
 2483                         /* Open a TCP direct Connection here, instead */ 
 2484  
 2485                                 if ((s = socket(AF_INET, SOCK_STREAM, 0)) == -1) {
 2486                                         pvmlogperror("mroute() socket");
 2487  
   ...
 2524                                                 }
 2525                                         }
 2526                                 }
 2527 #if !defined(NOUNIXDOM)
 2528                         }
 2529 #endif 
 2530  
 2531                         /* 1. Request direct connection  (TC_CONREQ)
 2532                          * 2. Request Notify so that we can close dead routes 
 2533                         */                       
 2534                         if (s >= 0) {  /* Socket was opened correctly */ 
 2535  
 2536                 /* Send Connection Request */ 
 2537                                 sbf = pvm_setsbuf(pvm_mkbuf(PvmDataFoo));
 2538                                 l = TDPROTOCOL;
 2539                                 pvm_pkint(&l, 1, 1);
 2540                                 pvm_pkstr(addr);
 2541                                 sbf = pvm_setsbuf(sbf);
 2542                                 up2 = midtobuf(sbf);
 2543                                 up2->m_dst = dtid;   /* Null Pointer Dereference (ID: 22102.29024) */
 2544                                 up2->m_tag = TC_CONREQ;
 2545                                 up2->m_ctx = SYSCTX_TC;
 2546                                 cc = mxfer(up2, &ztv);
 2547                                 pvm_freebuf(sbf);
 2548  
 2549                 /* now send a task exit notify to my pvmd */ 
 2550                                 if (cc >= 0) {
 2551                                         gotem += cc;
 2552                                         sbf = pvm_setsbuf(pvm_mkbuf(PvmDataFoo));
 2553                                         l = PvmTaskExit;
 2554                                         pvm_pkint(&l, 1, 1);
 2555                                         l = SYSCTX_TC;
 2556                                         pvm_pkint(&l, 1, 1);
 2557                                         l = TC_TASKEXIT;
 2558                                         pvm_pkint(&l, 1, 1);
 2559                                         l = 1;
 2560                                         pvm_pkint(&l, 1, 1);
 2561                                         pvm_pkint(&dtid, 1, 1);
 2562                                         sbf = pvm_setsbuf(sbf);
true2563                                         up2 = midtobuf(sbf);
up2 <= 40952564                                         up2->m_dst = TIDPVMD;     /* Null Pointer Dereference */
Preconditions 
mid <= pvmmidhsiz - 1
mid >= 1
dtid != 0
dtid != pvmmytid
ttlist->tt_link->tt_tid != dtid
pvmsbuf->m_enc != 536870912
alreadyhere = 0
numpmsgs != 0
pvmmidhsiz >= 2
pvmrouteopt = 3
td = 0
Postconditions 
freepmsgs.m_link->m_link->m_rlink' = freepmsgs.m_link->m_rlink
ttlist->tt_link->tt_rlink' = &$heap_337353
ttlist->tt_link->tt_rlink->tt_link' = &$heap_337353
freepmsgs.m_link->m_rlink->m_link' = freepmsgs.m_link->m_link
addr' = &$heap_337354
alreadyhere' = 1
((char*)&$unknown_1206478)[36]' = tag
((char*)&$unknown_1206478)[48]' = dtid
((char*)&$unknown_1206480)[32]' = 524287
((char*)&$unknown_1206480)[36]' = -2147287039
((char*)&$unknown_1206480)[48]' = dtid
cc' >= 0
f' = f + 1
gotem' = cc'
$heap_337352' is allocated by open
$heap_337352' is allocated
$heap_337353' = ttlist->tt_link
bytes_after(&$heap_337353)' = 64
$heap_337353' is allocated by malloc
$heap_337353' is allocated
bytes_before(&$heap_337353)' = 0
((char*)&$heap_337353)[16]' = &$heap_337352 - 4096
((char*)&$heap_337353)[4]' = ttlist->tt_link->tt_rlink
((char*)&$heap_337353)[52]' = freepmsgs.m_link
((char*)&$heap_337353)[60]' = &$heap_337354
((char*)&$heap_337353)[8]' = dtid
((char*)&$heap_337353)[12]' = 1
bytes_after(&$heap_337354)' = 35
$heap_337354' is allocated by malloc
bytes_before(&$heap_337354)' = 0
strlen(&$heap_337354)' = 34
tocttou($heap_337354)' = tocttou(spath)
oslen' = 110
pcbp' = &$heap_337353
s' = &$heap_337352 - 4096
sbf' <= 0
spath[0]' = 0
strlen(&spath[0])' = 34
spath[34]' = 0
td' = &#string21[0]
uns.sun_family' = 1
strlen(&uns.sun_family)' = 36
uns.sun_path[0]' = 0
tocttou(uns.sun_path)' = tocttou(spath)
up' = &$unknown_1206478
up2' = 0



Change Warning 22101.29023 : Null Pointer Dereference

Priority:
State:
Finding:
Owner:
Note: