Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Null Pointer Dereference  at pvmd.c:5265

Categories: LANG.MEM.NPD CWE:476
Warning ID: 2548.2579
Procedure: varsub
Trace: View
Modified: Wed Sep 2 12:40:55 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/pvmd.c
   Enter varsub
 5205 char * 
 5206 varsub(s) 
 5207         char *s;
 5208 {
 5209         int rm = 8;             /* length of result string space */ 
 5210         char *r;                /* result string */ 
true5211         int rl = 0;
 5212         char *p;
 5213         char *vn, *vv;
 5214         char c;
 5215         int l;
 5216  
rl <= 40955217         r = TALLOC(rm, char, "var");
 5218         while (*s) {   /* Null Pointer Dereference (ID: 2556.2587) */
 5219                 for (p = s; *p && *p != '$'; p++) ;
 5220                 if (l = p - s) {
 5221                         if (rl + l >= rm) {
 5222                                 rm = rl + l + 1;
 5223                                 r = TREALLOC(r, rm, char);
 5224                         }
 5225                         strncpy(r + rl, s, l);   /* Null Pointer Dereference (ID: 2549.2580) */
 5226                         rl += l;
 5227                 }
 5228                 s = p++;
 5229                 if (*s == '$') {
 5230                         if (*p == '{')
 5231                                 p++;
 5232                         vn = p;
 5233                         while (isalnum(*p) || *p == '_')   /* Buffer Overrun (ID: 2677.2726) */  /* Negative Character Value (ID: 2418.2435) */
 5234                                 p++;
 5235                         c = *p;
 5236                         *p = 0;
 5237  
 5238                         vv = getenv(vn);   /* Buffer Overrun (ID: 2676.2725) */
 5239  
 5240 #ifdef WIN32 
 5241                         if (!vv)
 5242                                 vv = (char *) read_pvmregistry(vn); 
 5243 #endif 
 5244  
 5245                         *p = c;
 5246                         if (*p == '}')
 5247                                 p++;
 5248                         if (vv)
 5249                                 l = strlen(vv);
 5250                         else {
 5251                                 vv = s;
 5252                                 l = p - s;
 5253                         }
 5254                         if (l) {
 5255                                 if (rl + l >= rm) {
 5256                                         rm = rl + l + 1;
 5257                                         r = TREALLOC(r, rm, char);
 5258                                 }
 5259                                 strncpy(r + rl, vv, l);   /* Buffer Overrun (ID: 2546.2577) */  /* Null Pointer Dereference (ID: 2547.2578) */
 5260                                 rl += l;
 5261                         }
 5262                         s = p;
 5263                 }
 5264         }
r <= 40955265         r[rl] = 0;     /* Null Pointer Dereference */
Preconditions 
&$unknown_482836 >= $param_1
*$param_1 != 0
*$param_1 != 36
strlen($param_1) != 0
$unknown_482836 != 0
$unknown_482836 != 36
strlen(&$unknown_482836) = 3
((char*)&$unknown_482836)[2] = 125
((char*)&$unknown_482836)[3] = 0
((char*)&$unknown_482836)[1] = 36
$input_12 = 0
Postconditions 
c' = ((char*)&$unknown_482836)[2]
$heap_32299' = $input_60
bytes_after(&$heap_32299)' = $input_12 + 1
$heap_32299' is allocated by malloc
$heap_32299' is a non-heap object
bytes_before(&$heap_32299)' = 0
strlen(&$heap_32299)' = $input_12
l' = $input_12
p' = &$unknown_482836 + 3
r' = 0
rl' = 0
rm' = 8
s' = &$unknown_482836 + 3
vn' = &$unknown_482836 + 2
vv' = &$heap_32299



Change Warning 2548.2579 : Null Pointer Dereference

Priority:
State:
Finding:
Owner:
Note: