Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Free Null Pointer  at tdpro.c:1916

Categories: LANG.ALLOC.FNP CWE:590
Warning ID: 2722.2775
Procedure: tm_mca
Trace: View
Modified: Wed Sep 2 12:44:25 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

Source  |  Language: C Hide Legend     
ProblemLineSource
   /u1/paul/SATE/2010/c/pvm/pvm3/src/tdpro.c
   Enter tm_mca
 1863 tm_mca(tp, mp) 
 1864         struct task *tp;
 1865         struct pmsg *mp;
 1866 {
 1867         struct mca *mcap;                       /* mca descriptor */ 
 1868         int ndst;                                       /* num of dst tids */ 
 1869         int *dsts;                                      /* dst tids */ 
 1870         int tid;
 1871         int i, j;
 1872  
 1873         /*
 1874         * unpack list of dst tids from message (expect it to be sorted).
 1875         * discard tids to nonexistent foreign hosts.
 1876         */ 
 1877  
 1878         mcap = mca_new();
 1879         mcap->mc_tid = TIDGID | tp->t_tid;   /* Null Pointer Dereference (ID: 2724.2777) */
 1880         upkint(mp, &ndst);
true1881         dsts = TALLOC(ndst, int, "dsts");
 1882         for ( i=0, j=0 ; i < ndst ; i++ ) {
 1883                 upkuint(mp, &tid);
 1884                 if ((tid & tidhmask) == myhostpart || tidtohost(hosts, tid))
 1885                         dsts[j++] = tid;   /* Null Pointer Dereference (ID: 2723.2776) */
 1886         }
 1887         ndst = j;
 1888  
 1889         if (ndst < 1)
 1890                 goto noguys;
 1891  
 1892         /*
 1893         * send DM_MCA messages containing tids to destination hosts 
 1894         * make list of destination hosts for us 
 1895         */ 
 1896  
 1897         mcap->mc_dsts = TALLOC(ndst, int, "mcal");      /* XXX cheat, too much space */ 
 1898         mcap->mc_ndst = 0;   /* Useless Assignment (ID: 2717.2770) */
 1899  
 1900         for (j = 0; j < ndst; ) {
   ...
 1905                 mp->m_dst = (tid |= TIDPVMD);   /* Null Pointer Dereference (ID: 2720.2773) */
 1906                 mp->m_tag = DM_MCA;
 1907                 pkint(mp, mcap->mc_tid);
 1908                 pkint(mp, j - i);
 1909                 while (i < j)
 1910                         pkint(mp, dsts[i++]);   /* Buffer Underrun (ID: 2719.2772) */
 1911                 sendmessage(mp);
 1912                 mcap->mc_dsts[mcap->mc_ndst++] = tid;   /* Null Pointer Dereference (ID: 2718.2771) */
 1913         }
 1914  
 1915 noguys: 
dsts <= 40951916         PVM_FREE(dsts);     /* Free Null Pointer */
Preconditions
hosts->ht_last >= 1
Postconditions
dsts' = 0
bytes_after(&$heap_161820)' = 32
$heap_161820' is allocated by malloc
$heap_161820' is allocated
bytes_before(&$heap_161820)' = 0
((char*)&$heap_161820)[20]' = 0
((char*)&$heap_161820)[24]' = 0
i' >= 2
j' <= 0
mcap' = &$heap_161820
mp' = $param_2
ndst' = j'
tp' = $param_1




Change Warning 2722.2775 : Free Null Pointer

Priority:
State:
Finding:
Owner:
Note: