Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Buffer Overrun  at host.c:910

Categories: LANG.MEM.BO CWE:120 CWE:121 CWE:122 CWE:126
Warning ID: 2609.2643
Procedure: readhostfile
Trace: View
Modified: Wed Sep 2 12:42:13 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/host.c
   Enter readhostfile
 820 struct htab * 
 821 readhostfile(fn) 
 822         char *fn;
 823 {
 824         struct htab *htp;
 825         struct hostd *hp;
 826         FILE *ff = 0;
 827         char buf[512];          /* line buffer */ 
 828         int lnum = 0;           /* line counter */ 
 829         int err = 0;            /* error count */ 
 830         struct hostent *he;
 831         char *p;
 832         struct in_addr *my_in_addrs;
 833         int num_addrs;
 834         int i;
 835         int maxhostid = tidhmask >> (ffs(tidhmask) - 1);
 836         struct hostd *defaults = 0;
 837  
 838         htp = ht_new(1);   /* Leak (ID: 2607.2641) */
 839  
 840         /*
 841         * get list of this host's interfaces so we can detect ourself in file 
 842         */ 
 843  
 844         if (iflist(&my_in_addrs, &num_addrs) == -1 || num_addrs < 1) {
 845                 pvmlogprintf("readhostfile() iflist failed\n");
 846                 goto bail;
 847         }
 848  
 849         if (!(ff = fopen(fn, "r"))) {
 850                 pvmlogprintf("readhostfile() %s: can't read\n", fn);
 851                 goto bail;
 852         }
 853  
 854         /* parse each line of host file */ 
 855  
 856         while (fgets(buf, sizeof(buf), ff)) {
 857                 lnum++;
 858                 for (p = buf; *p && isspace(*p); p++);   /* Negative Character Value (ID: 2410.2425) */
 859  
 860                 if (!*p || *p == '#')   /* leading '#' is comment */ 
 861                         continue;
 862  
 863                 if (lnum > maxhostid) {
 864                         pvmlogprintf("readhostfile() %s: too many lines\n", fn);
 865                         err++;
 866                         break;
 867                 }
 868                 hp = hd_new(lnum);
 869  
 870                 while (*p == '&' || *p == '$') {
 871                         if (*p == '&')          /* leading '&' is no-start */ 
 872                                 hp->hd_flag |= HF_NOSTART;   /* Null Pointer Dereference (ID: 2614.2648) */
 873                         if (*p == '$')          /* leading '$' allows overloading host */ 
 874                                 hp->hd_flag |= HF_OVERLOAD;   /* Null Pointer Dereference (ID: 2612.2646) */
 875                         p++;
 876                 }
 877  
 878                 if (parsehost(p, hp)) {
 879                         pvmlogprintf("readhostfile() %s %d: errors\n", fn, lnum);
 880                         err++;
 881                         goto badline;
 882                 }
 883  
 884         /*
 885         * if host == "*", change the default options 
 886         * instead of adding a new host to the list 
 887         */ 
 888  
 889                 if (!strcmp(hp->hd_name, "*")) {   /* Null Pointer Dereference (ID: 2611.2645) */
 890                         if (defaults)
 891                                 hd_unref(defaults);
 892                         defaults = hp;
 893                         continue;
 894                 }
 895  
 896         /* Set unspecified fields of hp to defaults */ 
 897  
 898                 if (defaults)
 899                         applydefaults(hp, defaults);
 900  
 901         /* look up ip addr */ 
 902  
true903                 if (!(he = gethostbyname(hp->hd_aname ? hp->hd_aname : hp->hd_name))) {
 904                         pvmlogprintf("readhostfile() %s %d: %s: can't gethostbyname\n",
 905                                 fn, lnum, hp->hd_name);
 906                         err++;
 907                         goto badline;
 908                 }
 909                 BCOPY(he->h_addr_list[0], (char*)&hp->hd_sad.sin_addr,
bytes_after(he) < 32910                         sizeof(struct in_addr));     /* Buffer Overrun */
 911  
 912         /* mark master host to not start */ 
 913  
 914                 if (!(hp->hd_flag & HF_OVERLOAD))
 915                         for (i = num_addrs; i-- > 0; ) {
 916                                 if (BCMP((char*)&my_in_addrs[i], (char*)&hp->hd_sad.sin_addr,
 917                                                 sizeof(struct in_addr)) == 0) {
 918                                         hp->hd_flag |= HF_NOSTART;
 919                                         break;
 920                                 }
 921                         }
 922  
 923                 ht_insert(htp, hp);
 924                 hd_unref(hp);
 925                 continue;
 926  
 927 badline: 
 928                 hd_unref(hp);
Preconditions 
&$unknown_772210 != 0
&$unknown_772211 >= &buf[0] + 1
$unknown_772211 <= 38
$unknown_772211 >= 36
strlen(&$unknown_772211) != 0
strlen(&$unknown_772211) != 1
((char*)&$unknown_772211)[1] != 0
((char*)&$unknown_772211)[1] != 36
((char*)&$unknown_772211)[1] != 38
((char*)&$heap_41780)[184] = 0
$input_500 <= 511
$input_500 >= 1
$input_508 = &$unknown_772210
Postconditions 
buf[0]' = &$unknown_772210
strlen(&buf[0])' = $input_500
defaults' = &$unknown_772213
ff' = &$heap_41779
fn' = $param_1
ghbn_h_name[0]' = 60
strlen(&ghbn_h_name[0])' = 255
ghbn_h_name[255]' = 0
he' = &stored_hostent.h_name
$heap_41777' = 0
bytes_after(&$heap_41777)' = 40
$heap_41777' is allocated by malloc
$heap_41777' is allocated
bytes_before(&$heap_41777)' = 0
strlen(&$heap_41777)' = 0
$heap_41778' is allocated by open
$heap_41778' is allocated
((char*)&$heap_41778)[-4096]' is open
$heap_41779' = &$heap_41778 - 4096
bytes_after(&$heap_41779)' = 1028
$heap_41779' is allocated by fopen
$heap_41779' is allocated
$heap_41779' is open
bytes_before(&$heap_41779)' = 0
$heap_41780' = 1
bytes_after(&$heap_41780)' = 200
$heap_41780' is allocated by malloc
$heap_41780' is allocated
bytes_before(&$heap_41780)' = 0
strlen(&$heap_41780)' = 0
((char*)&$heap_41780)[136]' = &$heap_41782
((char*)&$heap_41780)[152]' = 1
((char*)&$heap_41780)[176]' = 0
((char*)&$heap_41780)[88]' = 2
((char*)&$heap_41780)[104]' = 1
((char*)&$heap_41780)[108]' = 1
((char*)&$heap_41780)[112]' = &$heap_41783
((char*)&$heap_41780)[120]' = &$heap_41781
bytes_after(&$heap_41781)' = 184
$heap_41781' is allocated by malloc
$heap_41781' is allocated
bytes_before(&$heap_41781)' = 0
((char*)&$heap_41781)[16]' = 0
((char*)&$heap_41781)[176]' = 0
((char*)&$heap_41781)[24]' = 0
((char*)&$heap_41781)[32]' = 0
((char*)&$heap_41781)[40]' = 0
bytes_after(&$heap_41782)' = 184
$heap_41782' is allocated by malloc
$heap_41782' is allocated
bytes_before(&$heap_41782)' = 0
((char*)&$heap_41782)[16]' = 0
((char*)&$heap_41782)[176]' = 0
((char*)&$heap_41782)[24]' = 0
((char*)&$heap_41782)[32]' = 0
((char*)&$heap_41782)[40]' = 0
bytes_after(&$heap_41783)' = 184
$heap_41783' is allocated by malloc
$heap_41783' is allocated
bytes_before(&$heap_41783)' = 0
((char*)&$heap_41783)[16]' = 0
((char*)&$heap_41783)[176]' = 0
((char*)&$heap_41783)[24]' = 0
((char*)&$heap_41783)[32]' = 0
((char*)&$heap_41783)[40]' = 0
hp' = &$heap_41780
htp' = &$heap_41777
maxhostid' >= 2
num_addrs' >= 1
p' = &$unknown_772211 + 1



Change Warning 2609.2643 : Buffer Overrun

Priority:
State:
Finding:
Owner:
Note: