Text  |   XML   |   Visible Warnings:

Pvm : Pvm analysis 1 : Redundant Condition  at pmsg.c:1455

Categories: LANG.STRUCT.RC CWE:570 CWE:571
Warning ID: 2636.2676
Procedure: enc_xdr_ulong
Trace: View
Modified: Wed Sep 2 12:42:29 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/pmsg.c
   Enter enc_xdr_ulong
 1430 enc_xdr_ulong(mp, vp, cnt, std, siz) 
 1431         struct pmsg *mp;
 1432         void *vp;
 1433         int cnt, std, siz;
 1434 {
 1435         register unsigned long *np;
 1436         int cc = 0;
 1437  
 1438         for (np = (unsigned long*)vp; cnt-- > 0; np += std)
 1439 #ifdef USE_XDR_LONGLONG 
 1440                 if (!xdr_u_longlong_t(&mp->m_xdr, np)) 
 1441 #else 
 1442                 if (!xdr_u_long(&mp->m_xdr, np))
 1443 #endif 
 1444                 {
 1445 #ifdef USE_XDR_LONGLONG 
 1446                         if (*np & ~(long)0xffffffffffffffff) 
 1447 #else 
 1448                         if (*np & ~(long)0xffffffff)
 1449 #endif 
 1450                         {
 1451                                 cc = PvmOverflow;
 1452                                 break;
 1453                         }
 1454                         mp->m_frag->fr_rlink->fr_len = xdr_getpos(&mp->m_xdr);
Never True: cc != 01455                         if (cc = enc_xdr_step(mp))     /* Redundant Condition */
 1456                                 break;   /* Unreachable Control Flow (ID: 2637.2677) */
 1457                         else 
 1458 #ifdef USE_XDR_LONGLONG 
 1459                                 if (!xdr_u_longlong_t(&mp->m_xdr, np)) 
 1460 #else 
 1461                                 if (!xdr_u_long(&mp->m_xdr, np))
 1462 #endif 
 1463                                 {
 1464                                         cc = PvmNoMem;
 1465                                         break;
 1466                                 }
 1467                 }
 1468         mp->m_frag->fr_rlink->fr_len = xdr_getpos(&mp->m_xdr);
 1469         return cc;
 1470 } 



Change Warning 2636.2676 : Redundant Condition

Priority:
State:
Finding:
Owner:
Note: