Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Buffer Overrun  at pvmcruft.c:380

Categories: LANG.MEM.BO CWE:120 CWE:121 CWE:122 CWE:126
Warning ID: 2755.2810
Procedure: pvmget32
Trace: View
Modified: Wed Sep 2 12:45:03 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/lpvm.c
   Enter mxinput
 1545 mxinput(pcbp, nfr) 
 1546         struct ttpcb *pcbp;
 1547         int *nfr;                               /* number of frags read */ 
 1548 {
 1549         int gotem = 0;                  /* num msgs added to pvmrxlist */ 
 1550         struct frag *fp;                /* shadows pcbp->tt_rxf */ 
 1551         struct frag *fp2;
 1552         int n;                                  /* bytes received */ 
 1553         int m;                                  /* length of fragment */ 
 1554         struct pmsg *rxup;              /* message containing this frag */ 
 1555         struct pmsg *hdfrgpile;
 1556         char *cp;                               /* gp */ 
 1557         int src;
 1558         int dst;
 1559         int ff;
 1560         static int fairprobe = 0;
 1561  
 1562         *nfr = 0;
 1563  
 1564  
 1565 #if defined(IMA_MPP) 
 1566  
 1567         /* Messages from Tasks and  Messages from the daemon  are probed 
 1568          * fairprobe makes sure that neither interface gets starved.  
 1569          * fairprobe is toggled when an EOM is found   
 1570         */ 
 1571  
 1572         switch (fairprobe)
   ...
 1584                         if ((fp = pvm_readfrompvmd()) == (struct frag *) NULL)  
 1585                                 fp = pvm_readfrompeer();
 1586                         break;
 1587  
 1588         }
 1589  
 1590         if (!fp)  
 1591                 return gotem;   /* didn't read anything */ 
 1592  
 1593 #else 
 1594         if (!pcbp->tt_rxf)
 1595                 if (!(pcbp->tt_rxf = fr_new(pvmfrgsiz)))   /* Redundant Condition (ID: 2753.2808) */
 1596                         return PvmNoMem;   /* Unreachable Computation (ID: 2754.2809) */
 1597         fp = pcbp->tt_rxf;
 1598  
 1599         /* read fragment header separately from body */ 
 1600  
 1601         n = (fp->fr_len < TDFRAGHDR) ? 0 : pvmget32(fp->fr_dat + 8);
 1602         n += TDFRAGHDR - fp->fr_len;
 1603         if (pvmdebmask & PDMPACKET) {
 1604                 pvmlogprintf("mxinput() pcb t%x fr_len=%d fr_dat=+%d n=%d\n",
 1605                                 pcbp->tt_tid, fp->fr_len, fp->fr_dat - fp->fr_buf, n);
 1606         }
 1607 #ifndef WIN32 
 1608         n = read(pcbp->tt_fd, fp->fr_dat + fp->fr_len, n);   /* Null Pointer Dereference (ID: 2758.2813) */
 1609 #else 
 1610         n = win32_read_socket( pcbp->tt_fd,fp->fr_dat+fp->fr_len,n); 
 1611 #endif 
 1612         if (pvmdebmask & PDMPACKET) {
 1613                 pvmlogprintf("mxinput() read=%d\n", n);
 1614         }
 1615  
 1616         /* deal with errors and closes */ 
 1617  
 1618         if (n == -1 &&  
 1619 #ifndef WIN32 
 1620                         errno != EWOULDBLOCK &&  
 1621 #endif 
 1622                         errno != EINTR)
 1623         {
 1624                 if (pvmdebmask & PDMPACKET) {
 1625                         pvmlogperror("mxinput() read");
 1626                         pvmlogprintf("mxinput() t%x\n", pcbp->tt_tid);
 1627                 }
 1628                 return PvmSysErr;
 1629         }
 1630         if (!n) {
 1631                 if (pvmdebmask & PDMPACKET) {
 1632                         pvmlogprintf("mxinput() t%x read EOF\n", pcbp->tt_tid);
 1633                 }
 1634                 return -1;
 1635         }
 1636  
 1637         if (n < 1)
 1638                 return gotem;
 1639         if ((fp->fr_len += n) < TDFRAGHDR)
 1640                 return gotem;
 1641  
 1642         /* realloc buffer if frag won't fit */ 
 1643  
 1644         m = TDFRAGHDR + pvmget32(fp->fr_dat + 8);
true1645         if (fp->fr_len == TDFRAGHDR) {
 1646                 if (m > fp->fr_max - (fp->fr_dat - fp->fr_buf)) {
m < 401647                         fp2 = fr_new(m);
 1648                         BCOPY(fp->fr_dat, fp2->fr_dat, TDFRAGHDR);   /* Null Pointer Dereference (ID: 2757.2812) */
 1649                         fp2->fr_len = fp->fr_len;
 1650                         fr_unref(fp);
bytes_after(fp2->fr_dat) < 401651                         fp = pcbp->tt_rxf = fp2;
 1652                         if (pvmdebmask & PDMPACKET) {
 1653                                 pvmlogprintf("mxinput() realloc frag max=%d\n", m);
 1654                         }
 1655                 }
 1656         }
 1657  
 1658 #endif /* defined (IMA_MPP) */ 
 1659          
 1660         /* Now get information from the frag header to determine 
 1661          * src, destination, length, etc.
 1662         */ 
 1663 #if !defined(IMA_MPP)
 1664         if (fp->fr_len == m)  
 1665 #else 
 1666         if (fp)  /* got a frag */ 
 1667 #endif 
 1668         {
 1669                 (*nfr)++;
 1670 #if !defined (IMA_MPP)
 1671                 pcbp->tt_rxf = 0;
 1672 #endif 
bytes_after(fp->fr_dat) < 401673                 cp = fp->fr_dat;
 1674                 dst = pvmget32(cp);
 1675                 src = pvmget32(cp + 4);
 1676                 ff = pvmget8(cp + 12);
 1677                 fp->fr_len -= TDFRAGHDR;
 1678                 fp->fr_dat += TDFRAGHDR;
 1679  
 1680                 if (pvmdebmask & PDMPACKET) {
 1681                         pvmlogprintf("mxinput() pkt src t%x len %d ff %d\n",
 1682                                         src, fp->fr_len, ff);
 1683         }
 1684  
 1685 #if defined(IMA_MPP) 
 1686         if (fp -> fr_rip )      /* frag was received inplace */ 
 1687         {
 1688                 gotem++;
 1689                 fr_unref(fp); /* free frag -- mppprecv knows that the msg completed */ 
 1690                 return gotem;
 1691         }
 1692  
 1693         hdfrgpile = pvm_mpp_pmsgs(); 
 1694 #else 
 1695         hdfrgpile = pcbp->tt_rxfrag ;
 1696 #endif 
 1697         /*
 1698         * if start of message, make new umbuf, add to frag pile 
 1699         */ 
 1700                 if (ff & FFSOM) {
bytes_after(cp) < 401701                         cp += TDFRAGHDR;
 1702                         fp->fr_len -= MSGHDRLEN;
 1703                         fp->fr_dat += MSGHDRLEN;
 1704                         rxup = umbuf_new();
 1705                         rxup->m_enc = pvmget32(cp);   /* Null Pointer Dereference (ID: 2756.2811) */
 1706                         rxup->m_tag = pvmget32(cp + 4);
 1707                         rxup->m_ctx = pvmget32(cp + 8);
 1708                         rxup->m_wid = pvmget32(cp + 16);
bytes_after(cp) < 241709                         rxup->m_crc = pvmget32(cp + 20);
     /u1/paul/SATE/2010/c/pvm/pvm3/src/pvmcruft.c
     Enter mxinput / pvmget32
bytes_after($param_1) < 4374   pvmget32(p) 
 375           char *p;
 376   {
 377           return ((0xff & p[0]) << 24) 
 378           + ((0xff & p[1]) << 16) 
 379           + ((0xff & p[2]) << 8) 
bytes_after(p) < 4380           + (0xff & p[3]);     /* Buffer Overrun */  /* Null Pointer Dereference (ID: 2497.2527) */
     Exit mxinput / pvmget32
Preconditions 
&$unknown_1230280 <= -1 * ((char*)&((char*)*$param_1)[72])[36] + 16
&$unknown_1230280 >= 1
((char*)$param_1)[24] >= 0
((char*)&((char*)*$param_1)[72])[36] <= 15
((char*)&((char*)*$param_1)[72])[40] = 1
((char*)&((char*)*$param_1)[72])[42] != 0
numfrags = 1
numpmsgs != 0
$input_12 = &$unknown_1230280
$input_12 >= 1
Postconditions 
freefrags.fr_link->fr_link->fr_rlink' = freefrags.fr_link->fr_rlink
freepmsgs.m_link->m_link->m_rlink' = freepmsgs.m_link->m_rlink
freefrags.fr_link->fr_buf' = &$heap_164067 + 16
freefrags.fr_link->fr_dat' = &$heap_164067 + 64
freefrags.fr_link->fr_max' = 16
freefrags.fr_link->fr_len' = -32
freefrags.fr_link->fr_u.ref' = 1
freefrags.fr_link->fr_u.dab' = 1
freefrags.fr_link->fr_u.spr' = 0
freefrags.fr_link->fr_rip' = 0
freefrags.fr_link->fr_rlink' = 0
freepmsgs.m_link->m_codef' = 0
freepmsgs.m_link->m_cfrag' = 0
freepmsgs.m_link->m_ref' = 1
freepmsgs.m_link->m_len' = 0
freepmsgs.m_link->m_src' = 0
freepmsgs.m_link->m_dst' = 0
freepmsgs.m_link->m_flag' = 0
freepmsgs.m_link->m_cpos' = 0
freepmsgs.m_link->m_rlink' = 0
freepmsgs.m_link->m_crc' = 0
freefrags.fr_link->fr_rlink->fr_link' = freefrags.fr_link->fr_link
freepmsgs.m_link->m_rlink->m_link' = freepmsgs.m_link->m_link
freefrags.fr_link->fr_link' = 0
freepmsgs.m_link->m_link' = 0
*stderr' is allocated by fopen
bytes_before(stderr)' = 0
cp' = &$heap_164067 + 32
errno' != 0
fp' = freefrags.fr_link
fp2' = freefrags.fr_link
gotem' = 0
hdfrgpile' = ((char*)$param_1)[64]
bytes_after(&$heap_164067)' = 32
$heap_164067' is allocated by malloc
$heap_164067' is allocated
bytes_before(&$heap_164067)' = 0
((char*)&$heap_164067)[12]' = 1
m' = 16
n' = &$unknown_1230280
newline' = 1
nfr' = $param_2
numpmsgs' = numpmsgs - 1
p' = &$heap_164067 + 52
pcbp' = $param_1
rxup' = freepmsgs.m_link



Change Warning 2755.2810 : Buffer Overrun

Priority:
State:
Finding:
Owner:
Note: