Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Double Close  at lpvm.c:3428

Categories: POSIX.IO.DC CWE:672
Warning ID: 2765.2820
Procedure: pvmbeatask
Trace: View
Modified: Wed Sep 2 12:45:49 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 pvmbeatask
 2999 pvmbeatask() 
 3000 {
 3001         int sbf = 0, rbf = 0;                   /* saved rx and tx message handles */ 
 3002         int prver;                                              /* protocol version */ 
 3003         int cookie;                                             /* cookie assigned by pvmd for ident */ 
 3004         int cc;
 3005         char **ep=0;
 3006         char authfn[PVMTMPNAMLEN];      /* auth file name */ 
 3007         int authfd = -1;                                /* auth fd to validate pvmd ident */ 
 3008         int i;
 3009         char buf[16];                                   /* for converting sockaddr */ 
 3010         char *p;
 3011         struct pvmminfo minfo;
 3012         int outtid, outctx, outtag;
 3013         int trctid, trcctx, trctag;
 3014         int need_trcinfo = 0;
 3015         int new_tracer = 0;
 3016         char tmask[ 2 * TEV_MASK_LENGTH ];
 3017         int tbuf, topt;
 3018         int mid;
 3019 #ifdef IMA_BEOLIN 
 3020         struct hostent *hostaddr;
 3021         char namebuf[128]; 
 3022 #endif 
 3023         TEV_DECLS 
 3024 #ifdef WIN32 
 3025  
 3026    /*   WSAStartup has to be called before any socket command 
 3027                 can be executed. Why ? Ask Bill 
 3028    */ 
 3029         for (ep=environ;*ep;ep++)
 3030                 pvmputenv(*ep);    
 3031    if (WSAStartup(0x0101, &WSAData) != 0)
 3032         {
 3033       fprintf(stderr,"\nWSAStartup() failed\n");
 3034       ExitProcess(1);
 3035         } 
 3036         /* setsockopt(INVALID_SOCKET,SOL_SOCKET,SO_OPENTYPE, */ 
 3037         /*      (char *)&nAlert,sizeof(int)); */ 
 3038  
 3039 #endif 
 3040 /*
 3041         pvm_setopt(PvmDebugMask, -1);
 3042 */ 
 3043         if (pvmmytid != -1)
 3044                 return 0;
 3045  
 3046         pvmmydsig = pvmgetdsig();
 3047  
 3048         TEV_EXCLUSIVE;
 3049  
 3050         authfn[0] = 0;
 3051  
 3052 #ifndef WIN32 
 3053         pvmmyupid = getpid();
 3054 #else 
 3055         pvmmyupid = _getpid();     /* .... */ 
 3056 #endif 
 3057  
 3058         /*
 3059         * get expected pid from environment in case we were started by 
 3060         * the pvmd and someone forked again 
 3061         */ 
 3062  
 3063         if (p = getenv("PVMEPID"))
 3064                 cookie = atoi(p);
 3065         else 
 3066                 cookie = 0;
 3067  
 3068         /* if ((pvm_useruid = getuid()) == -1) */ 
 3069  
 3070 #ifndef WIN32 
 3071         if ((pvm_useruid = geteuid()) == -1) {
 3072                 pvmlogerror("can't getuid()\n");
 3073                 cc = PvmSysErr;
 3074                 goto bail2;
 3075         }
 3076         pvmchkuid( pvm_useruid );
 3077 #else 
 3078         if (!username)
 3079                 username = MyGetUserName(); 
 3080 #endif 
 3081  
 3082         if (p = getenv("PVMTASKDEBUG")) {
 3083                 pvmdebmask = pvmstrtoi(p);
 3084                 if (pvmdebmask) {
 3085                         pvmlogprintf("task debug mask is 0x%x (%s)\n",
 3086                                         pvmdebmask, debug_flags(pvmdebmask));
 3087                 }
 3088         }
 3089  
 3090 #ifndef IMA_MPP 
 3091         if (cc = mksocs())              /* make socket to talk to pvmd */ 
 3092                 goto bail2;
 3093 #endif 
 3094  
 3095         /*
 3096         *       initialize received-message list 
 3097         */ 
 3098  
 3099         pvmrxlist = pmsg_new(1);
 3100         BZERO((char*)pvmrxlist, sizeof(struct pmsg));   /* Null Pointer Dereference (ID: 2768.2823) */
 3101         pvmrxlist->m_link = pvmrxlist->m_rlink = pvmrxlist;
 3102  
 3103         ttlist = TALLOC(1, struct ttpcb, "tpcb");
 3104         BZERO((char*)ttlist, sizeof(struct ttpcb));   /* Null Pointer Dereference (ID: 2767.2822) */
 3105         ttlist->tt_link = ttlist->tt_rlink = ttlist;
 3106  
 3107         sbf = pvm_setsbuf(pvm_mkbuf(PvmDataFoo));
 3108         rbf = pvm_setrbuf(0);
 3109  
 3110         /*
 3111         *       create empty t-auth file that pvmd must write and we'll check later 
 3112         */ 
 3113  
 3114 #ifndef IMA_MPP 
 3115 #ifndef NOPROT 
 3116         (void)PVMTMPNAMFUN(authfn);
 3117 #ifdef IMA_OS2 
 3118         if ((authfd = open(authfn, O_RDWR|O_CREAT|O_EXCL|O_TRUNC, 0600)) == -1) 
 3119 #else 
 3120         if ((authfd = open(authfn, O_RDONLY|O_CREAT|O_EXCL|O_TRUNC, 0600)) == -1)
 3121 #endif 
 3122         {
 3123                 pvmlogperror(authfn);
 3124                 pvmlogerror("pvmbeatask() can't creat t-auth file\n");
 3125                 cc = PvmSysErr;
 3126                 goto bail2;
 3127         }
 3128 #endif 
 3129  
 3130         /*
 3131         *       send first connect message to pvmd 
 3132         */ 
 3133  
 3134         mxfersingle = 1;
 3135         prver = TDPROTOCOL;
 3136         pvm_pkint(&prver, 1, 1);
 3137         pvm_pkstr(authfn);
 3138         if ((cc = msendrecv(TIDPVMD, TM_CONNECT, SYSCTX_TM)) <= 0)
 3139                 goto bail;
 3140         pvm_upkint(&prver, 1, 1);
 3141         if (prver != TDPROTOCOL) {
 3142                 pvmlogprintf("pvmbeatask() t-d protocol mismatch (%d/%d)\n",
 3143                         TDPROTOCOL, prver);
 3144                 cc = PvmSysErr;
 3145                 goto bail;
 3146         }
 3147         pvm_upkint(&cc, 1, 1);
 3148         if (!cc) {
 3149                 pvmlogerror("pvmbeatask() pvmd refuses connection\n");
 3150                 cc = PvmSysErr;
 3151                 goto bail;
 3152         }
 3153  
 3154         /*
 3155         *       check our t-auth file; write in pvmd d-auth file 
 3156         */ 
 3157  
 3158 #ifndef NOPROT 
 3159  
 3160 #ifdef IMA_BEOLIN 
 3161         sleep(15);  /* because of race condition over NFS--yuck! (PLS) */ 
 3162 #endif 
 3163  
 3164         if ((cc = read(authfd, (char*)&cc, 1)) == -1) {
 3165                 pvmlogperror("pvmbeatask() read authfile");
 3166                 cc = PvmSysErr;
 3167                 goto bail;
 3168         }
 3169         if (cc != 1) {
 3170                 pvmlogerror("pvmbeatask() pvmd didn't validate itself\n");
 3171                 cc = PvmSysErr;
 3172                 goto bail;
 3173         }
 3174         (void)close(authfd);
 3175         (void)unlink(authfn);
 3176  
 3177         pvm_upkstr(authfn);
 3178         if ((authfd = open(authfn, O_WRONLY, 0)) == -1) {   /* File System Race Condition (ID: 2766.2821) */
 3179                 pvmlogperror(authfn);
 3180                 pvmlogerror("pvmbeatask() failed to open d-auth file\n");
 3181                 authfn[0] = 0;
 3182                 cc = PvmSysErr;
 3183                 goto bail;
 3184         }
 3185         cc = write(authfd, authfn, 1);
true3186         (void)close(authfd);
 3187         if (cc != 1) {
 3188                 if (cc == -1)
 3189                         pvmlogperror(authfn);
 3190                 pvmlogerror("pvmbeatask() can't write d-auth file\n");
 3191                 authfn[0] = 0;
 3192                 cc = PvmSysErr;
 3193                 goto bail;
 3194         }
 3195         authfd = -1;
 3196         authfn[0] = 0;
 3197  
 3198 #endif 
 3199  
 3200         /*
 3201         *       send second connect message to pvmd 
 3202         */ 
 3203  
   ...
 3408                         pvm_setrbuf(rbf);
 3409         }
 3410  
 3411         tev_init();
 3412  
 3413         if (TEV_AMEXCL) {
 3414                 TEV_ENDEXCL;
 3415         }
 3416         return 0;
 3417  
 3418 bail: 
 3419         if (pvm_getrbuf() > 0)
 3420                 pvm_freebuf(pvm_getrbuf());
 3421         if (pvm_getsbuf() > 0)
 3422                 pvm_freebuf(pvm_getsbuf());
 3423         pvm_setrbuf(rbf);
 3424         pvm_setsbuf(sbf);
 3425  
 3426 #ifndef IMA_MPP 
 3427         if (authfd != -1)
*authfd is closed3428                 (void)close(authfd);     /* Double Close */
Preconditions 
&$unknown_1444294 != 0
&$unknown_1444301 >= 0
&$unknown_1444316 >= 1
((char*)&$unknown_1444321)[44] >= 1
((char*)&$unknown_1444328)[44] >= 1
((char*)&$heap_172444)[1] <= 120
((char*)&$heap_172444)[1] >= 48
myfmt = -1
pvmmytid = -1
pvmtoplvl = 0
$input_12 >= 1
$input_148508 >= 0
$input_148516 = 1
$input_60 = 48
Postconditions 
((char*)$unknown_1444297)[8]' = ((char*)&$unknown_1444297)[8]
((char*)$unknown_1444297)[8]' = $unknown_1444297
*stderr' is allocated by fopen
bytes_before(stderr)' = 0
strlen(&$unknown_1444297)' = 0
authfd' = &$heap_172448 - 4096
authfn[0]' = 0
authfn[0]' is opened
strlen(&authfn[0])' = 0
authfn[0]' is checked by path name
strlen(&buf[0])' = 0
cc' = -14
cookie' = 0
ep' = 0
f' = f + 1
$heap_172444' = $input_60
bytes_after(&$heap_172444)' = $input_12 + 1
$heap_172444' is allocated by malloc
$heap_172444' is a non-heap object
bytes_before(&$heap_172444)' = 0
strlen(&$heap_172444)' = $input_12
$heap_172445' = &$heap_172445
bytes_after(&$heap_172445)' = 88
$heap_172445' is allocated by malloc
$heap_172445' is allocated
bytes_before(&$heap_172445)' = 0
strlen(&$heap_172445)' = 0
((char*)&$heap_172445)[8]' = &$heap_172445
bytes_after(&$heap_172446)' = $input_148508 + 1
$heap_172446' is allocated by malloc
$heap_172446' is a non-heap object
bytes_before(&$heap_172446)' = 0
strlen(&$heap_172446)' = $input_148508
$heap_172447' is allocated by open
$heap_172448' is allocated by open
$heap_172448' is freed
((char*)&$heap_172448)[-4096]' is closed
mxfersingle' = 1
need_trcinfo' = 0
new_tracer' = 0
newline' = 0
nextwaitid' = nextwaitid + 1
p' = &$heap_172444
prver' = 1318
pvm_useruid' != -1
pvmampushed' = 0
pvmamtraced' = pvmtoplvl
pvmdebmask' = &$unknown_1444294
pvmmidhsiz' >= 2
pvmmidhsiz' >= ((char*)&$unknown_1444321)[44] + 1
pvmrxlist' = &$unknown_1444297
rbf' = pvmrbuf->m_mid
sbf' = &$unknown_1444332
td' = &$heap_172446
ttlist' = &$heap_172445



Change Warning 2765.2820 : Double Close

Priority:
State:
Finding:
Owner:
Note: