2024-03-25 08:55 ftp://ftp.de.debian.org/debian/pool/main/q/qrouter/qrouter_1.4.71.orig.tar.gz cppcheck-options: --library=posix --library=gnu --showtime=top5 --check-library --inconclusive --enable=style,information --inline-suppr --disable=missingInclude --suppress=unmatchedSuppression --template=daca2 --debug-warnings --suppress=autoNoType --suppress=valueFlowBailout --suppress=bailoutUninitVar --suppress=symbolDatabaseWarning --suppress=valueFlowBailoutIncompleteVar -D__GNUC__ --platform=unix64 -j4 platform: Linux-6.1.0-0.deb11.6-amd64-x86_64-with-glibc2.31 python: 3.9.2 client-version: 1.3.45 compiler: g++ (Debian 10.2.1-6) 10.2.1 20210110 cppcheck: head 2.13.0 head-info: fd7bd50 (2024-03-25 07:57:46 +0100) count: 180 178 elapsed-time: 17.6 40.1 head-timing-info: Tokenizer::simplifyTokens1::ValueFlow: 15.7465s (avg. 15.7465s - 1 result(s)) valueFlowLifetime(tokenlist, errorLogger, settings): 8.20528s (avg. 2.05132s - 4 result(s)) valueFlowSymbolic(tokenlist, symboldatabase, errorLogger, settings): 5.15163s (avg. 5.15163s - 1 result(s)) valueFlowUninit(tokenlist, errorLogger, settings): 1.02802s (avg. 0.342673s - 3 result(s)) valueFlowCondition(SimpleConditionHandler{}, tokenlist, symboldatabase, errorLogger, settings, skippedFunctions): 0.651149s (avg. 0.21705s - 3 result(s)) Overall time: 15.8663s old-timing-info: Tokenizer::simplifyTokens1::ValueFlow: 38.4986s (avg. 38.4986s - 1 result(s)) valueFlowLifetime(tokenlist, errorLogger, settings): 29.1845s (avg. 7.29613s - 4 result(s)) valueFlowSymbolic(tokenlist, symboldatabase, settings): 5.26585s (avg. 5.26585s - 1 result(s)) valueFlowUninit(tokenlist, settings): 1.76689s (avg. 0.588963s - 3 result(s)) valueFlowCondition(SimpleConditionHandler{}, tokenlist, symboldatabase, errorLogger, settings, skippedFunctions): 0.929292s (avg. 0.309764s - 3 result(s)) Overall time: 38.6105s head results: qrouter-1.4.71/main.c:65:39: style: Parameter 'lastlayer' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/graphics.c:299:27: error: Division by zero. [zerodiv] qrouter-1.4.71/graphics.c:292:14: note: Assignment 'maxval=0', assigned value is 0 qrouter-1.4.71/graphics.c:293:19: note: Assuming condition is false qrouter-1.4.71/graphics.c:299:27: note: Division by zero qrouter-1.4.71/graphics.c:60:13: style: Variable 'Pr' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/graphics.c:90:13: style: Variable 'Pr' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/graphics.c:128:13: style: Variable 'Pr' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/graphics.c:777:47: style: Parameter 'interp' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/graphics.c:778:11: style: Parameter 'objv' can be declared as const array [constParameter] qrouter-1.4.71/mask.c:136:10: style: Unused variable: j [unusedVariable] qrouter-1.4.71/def.c:320:12: warning: Identical inner 'if' condition is always true. [identicalInnerCondition] qrouter-1.4.71/def.c:316:15: note: outer condition: lefl!=NULL qrouter-1.4.71/def.c:320:12: note: identical inner condition: lefl!=NULL qrouter-1.4.71/def.c:1173:7: error: Width 2047 given in format string (no. 1) is larger than destination buffer 'pinname[1024]', use %1023s to prevent overflowing it. [invalidScanfFormatWidth] qrouter-1.4.71/def.c:790:2: error: Common realloc mistake: 'Nlnets' nulled but not freed upon failure [memleakOnRealloc] qrouter-1.4.71/def.c:2074:9: style: Variable 'token' is reassigned a value before the old one has been used. [redundantAssignment] qrouter-1.4.71/def.c:2073:9: note: token is assigned qrouter-1.4.71/def.c:2074:9: note: token is overwritten qrouter-1.4.71/def.c:2075:9: style: Variable 'token' is reassigned a value before the old one has been used. [redundantAssignment] qrouter-1.4.71/def.c:2074:9: note: token is assigned qrouter-1.4.71/def.c:2075:9: note: token is overwritten qrouter-1.4.71/def.c:46:19: style: Parameter 'name' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/def.c:63:18: style: Parameter 'name' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/def.c:613:75: style: Parameter 'home' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/def.c:1959:15: style: Parameter 'inName' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/def.c:1966:9: style: Unused variable: i [unusedVariable] qrouter-1.4.71/lef.c:3646:11: style: Condition 'lefl' is always true [knownConditionTrueFalse] qrouter-1.4.71/lef.c:3219:2: warning: fprintf format string requires 0 parameters but 1 is given. [wrongPrintfScanfArgNum] qrouter-1.4.71/lef.c:3054:9: style: Variable 'token' is reassigned a value before the old one has been used. [redundantAssignment] qrouter-1.4.71/lef.c:3053:9: note: token is assigned qrouter-1.4.71/lef.c:3054:9: note: token is overwritten qrouter-1.4.71/lef.c:3066:9: style: Variable 'token' is reassigned a value before the old one has been used. [redundantAssignment] qrouter-1.4.71/lef.c:3065:9: note: token is assigned qrouter-1.4.71/lef.c:3066:9: note: token is overwritten qrouter-1.4.71/lef.c:407:31: style:inconclusive: Function 'LefSkipSection' argument 2 names different: declaration 'match' definition 'section'. [funcArgNamesDifferent] qrouter-1.4.71/lef.h:151:37: note: Function 'LefSkipSection' argument 2 names different: declaration 'match' definition 'section'. qrouter-1.4.71/lef.c:407:31: note: Function 'LefSkipSection' argument 2 names different: declaration 'match' definition 'section'. qrouter-1.4.71/lef.c:3114:18: warning:inconclusive: Found suspicious equality comparison. Did you intend to assign a value instead? [constStatement] qrouter-1.4.71/lef.c:448:19: style: Parameter 'name' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/lef.c:572:20: style: Parameter 'token' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/lef.c:1411:11: style: Variable 'token' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/lef.c:1486:11: style: Variable 'token' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/lef.c:1802:11: style: Variable 'token' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/lef.c:1966:11: style: Parameter 'pinName' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/lef.c:2083:16: style: Variable 'lef_class_to_bitmask' can be declared as const array [constVariable] qrouter-1.4.71/lef.c:2106:16: style: Variable 'lef_use_to_bitmask' can be declared as const array [constVariable] qrouter-1.4.71/lef.c:2195:11: style: Variable 'token' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/lef.c:2519:27: style: Parameter 'name' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/lef.c:2543:25: style: Parameter 'name' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/lef.c:3486:11: style: Parameter 'inName' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/lef.c:540:28: error: Uninitialized struct member: newlefl.lefName [uninitStructMember] qrouter-1.4.71/lef.c:755:12: style: Unused variable: o [unusedVariable] qrouter-1.4.71/lef.c:770:12: style: Unused variable: o [unusedVariable] qrouter-1.4.71/lef.c:1247:10: style: Unused variable: lrect [unusedVariable] qrouter-1.4.71/lef.c:1249:12: style: Unused variable: width [unusedVariable] qrouter-1.4.71/lef.c:2621:19: style: Unused variable: tp [unusedVariable] qrouter-1.4.71/lef.c:2626:13: style: Unused variable: altVia [unusedVariable] qrouter-1.4.71/lef.c:3492:18: style: Unused variable: layer [unusedVariable] qrouter-1.4.71/lef.c:3495:12: style: Unused variable: xydiff [unusedVariable] qrouter-1.4.71/qconfig.c:249:18: style: Array index 'i' is used before limits check. [arrayIndexThenCheck] qrouter-1.4.71/qconfig.c:238:15: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:260:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:371:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:376:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:381:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:411:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:437:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:464:18: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:473:11: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/qconfig.c:191:23: style:inconclusive: Function 'read_config' argument 1 names different: declaration 'configfileptr' definition 'fconfig'. [funcArgNamesDifferent] qrouter-1.4.71/qconfig.h:49:24: note: Function 'read_config' argument 1 names different: declaration 'configfileptr' definition 'fconfig'. qrouter-1.4.71/qconfig.c:191:23: note: Function 'read_config' argument 1 names different: declaration 'configfileptr' definition 'fconfig'. qrouter-1.4.71/qconfig.c:96:12: style: Unused variable: h [unusedVariable] qrouter-1.4.71/qconfig.c:96:15: style: Unused variable: v [unusedVariable] qrouter-1.4.71/qconfig.c:170:15: style: Unused variable: strl [unusedVariable] qrouter-1.4.71/qconfig.c:473:9: style: Variable 'i' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/qconfig.c:200:12: style: Unused variable: dnr [unusedVariable] qrouter-1.4.71/qconfig.c:201:12: style: Unused variable: cn [unusedVariable] qrouter-1.4.71/qconfig.c:202:12: style: Unused variable: strl [unusedVariable] qrouter-1.4.71/qrouter.c:1610:10: style: The if condition is the same as the previous if condition [duplicateCondition] qrouter-1.4.71/qrouter.c:1609:10: note: First condition qrouter-1.4.71/qrouter.c:1610:10: note: Second condition qrouter-1.4.71/qrouter.c:1611:10: style: The if condition is the same as the previous if condition [duplicateCondition] qrouter-1.4.71/qrouter.c:1610:10: note: First condition qrouter-1.4.71/qrouter.c:1611:10: note: Second condition qrouter-1.4.71/qrouter.c:1035:22: style: Condition 'FailedNets!=(struct netlist_NETLIST)NULL' is always true [knownConditionTrueFalse] qrouter-1.4.71/qrouter.c:1528:22: style: Condition 'FailedNets!=(struct netlist_NETLIST)NULL' is always true [knownConditionTrueFalse] qrouter-1.4.71/qrouter.c:1607:10: style: Condition 'net' is always true [knownConditionTrueFalse] qrouter-1.4.71/qrouter.c:1354:6: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] qrouter-1.4.71/qrouter.c:1508:6: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] qrouter-1.4.71/qrouter.c:2105:9: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] qrouter-1.4.71/qrouter.c:2278:5: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] qrouter-1.4.71/qrouter.c:249:10: style: Variable 'Filename' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/qrouter.c:260:13: style: Variable 'optarg' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/qrouter.c:414:9: style: Variable 'layername' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/qrouter.c:943:20: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/qrouter.c:1684:35: style: Parameter 'netname' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/qrouter.c:1504:22: error: Uninitialized variable: failcount [uninitvar] qrouter-1.4.71/qrouter.c:1497:24: note: Assuming condition is false qrouter-1.4.71/qrouter.c:1504:22: note: Uninitialized variable: failcount qrouter-1.4.71/qrouter.c:537:11: style: Unused variable: rt [unusedVariable] qrouter-1.4.71/qrouter.c:538:9: style: Unused variable: seg [unusedVariable] qrouter-1.4.71/qrouter.c:987:12: style: Unused variable: nl [unusedVariable] qrouter-1.4.71/qrouter.c:1707:8: style: Unused variable: node [unusedVariable] qrouter-1.4.71/qrouter.c:1708:8: style: Unused variable: i [unusedVariable] qrouter-1.4.71/qrouter.c:1708:11: style: Unused variable: j [unusedVariable] qrouter-1.4.71/qrouter.c:1970:10: style: Variable 'netnum' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/qrouter.c:1802:17: style: Unused variable: dir [unusedVariable] qrouter-1.4.71/qrouter.c:1805:12: style: Unused variable: lnode [unusedVariable] qrouter-1.4.71/qrouter.c:2011:26: style: Unused variable: newpt [unusedVariable] qrouter-1.4.71/tclqrouter.c:2030:11: warning: Opposite inner 'if' condition leads to a dead code block. [oppositeInnerCondition] qrouter-1.4.71/tclqrouter.c:2029:34: note: outer condition: cn->next qrouter-1.4.71/tclqrouter.c:2030:11: note: opposite inner condition: !cn->next qrouter-1.4.71/tclqrouter.c:971:11: style: Condition 'net!=NULL' is always true [knownConditionTrueFalse] qrouter-1.4.71/tclqrouter.c:1352:11: style: Condition 'net!=NULL' is always true [knownConditionTrueFalse] qrouter-1.4.71/tclqrouter.c:1783:22: style: Condition 'outfile==NULL' is always true [knownConditionTrueFalse] qrouter-1.4.71/tclqrouter.c:1779:21: note: Assignment 'outfile=NULL', assigned value is 0 qrouter-1.4.71/tclqrouter.c:1783:22: note: Condition 'outfile==NULL' is always true qrouter-1.4.71/tclqrouter.c:249:24: style: Parameter 'f' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/tclqrouter.c:327:25: style: Parameter 'f' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/tclqrouter.c:360:11: style: Variable 'postcmd' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:360:48: style: Variable 'sres' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:394:13: style: Variable 'tkpath' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:552:10: style: Variable 'nullgvar' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:1651:11: style: Variable 'argv' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:1731:11: style: Variable 'option' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:1823:11: style: Variable 'configname' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:2239:11: style: Variable 'vname' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tclqrouter.c:868:23: style: Unused variable: saveOverhead [unusedVariable] qrouter-1.4.71/tclqrouter.c:1044:23: style: Unused variable: saveOverhead [unusedVariable] qrouter-1.4.71/tclqrouter.c:1233:23: style: Unused variable: saveOverhead [unusedVariable] qrouter-1.4.71/tclqrouter.c:1732:19: style: Variable 'do_fix' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/tclqrouter.c:2456:44: style: Unused variable: pinname [unusedVariable] qrouter-1.4.71/tclqrouter.c:276:4: error: va_list 'args' used before va_start() was called. [va_list_usedBeforeStarted] qrouter-1.4.71/tclqrouter.c:277:46: error: va_list 'args' used before va_start() was called. [va_list_usedBeforeStarted] qrouter-1.4.71/tclqrouter.c:278:4: error: va_list 'args' used before va_start() was called. [va_list_usedBeforeStarted] qrouter-1.4.71/tclqrouter.c:281:7: error: va_list 'args' used before va_start() was called. [va_list_usedBeforeStarted] qrouter-1.4.71/tclqrouter.c:285:47: error: va_list 'args' used before va_start() was called. [va_list_usedBeforeStarted] qrouter-1.4.71/tclqrouter.c:286:7: error: va_list 'args' used before va_start() was called. [va_list_usedBeforeStarted] qrouter-1.4.71/tkSimple.c:124:11: style: Variable 'arg' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tkSimple.c:270:9: style: Variable 'arg' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/tkSimple.c:423:19: style: Variable 'configEventPtr' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/maze.c:2808:17: style: The if condition is the same as the previous if condition [duplicateCondition] qrouter-1.4.71/maze.c:2801:17: note: First condition qrouter-1.4.71/maze.c:2808:17: note: Second condition qrouter-1.4.71/maze.c:2764:10: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] qrouter-1.4.71/maze.c:2764:10: portability: %d in format string (no. 2) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] qrouter-1.4.71/maze.c:1200:11: warning: Either the condition 'rt==NULL' is redundant or there is possible null pointer dereference: rt. [nullPointerRedundantCheck] qrouter-1.4.71/maze.c:1216:16: note: Assuming that condition 'rt==NULL' is not redundant qrouter-1.4.71/maze.c:1200:11: note: Null pointer dereference qrouter-1.4.71/maze.c:548:17: style: The comparison 'result == 0' is always true. [knownConditionTrueFalse] qrouter-1.4.71/maze.c:362:18: note: 'result' is assigned value '0' here. qrouter-1.4.71/maze.c:548:17: note: The comparison 'result == 0' is always true. qrouter-1.4.71/maze.c:358:36: style:inconclusive: Function 'set_node_to_net' argument 2 names different: declaration 'newnet' definition 'newflags'. [funcArgNamesDifferent] qrouter-1.4.71/maze.h:11:40: note: Function 'set_node_to_net' argument 2 names different: declaration 'newnet' definition 'newflags'. qrouter-1.4.71/maze.c:358:36: note: Function 'set_node_to_net' argument 2 names different: declaration 'newnet' definition 'newflags'. qrouter-1.4.71/maze.c:764:47: style:inconclusive: Function 'set_routes_to_net' argument 3 names different: declaration 'newnet' definition 'newflags'. [funcArgNamesDifferent] qrouter-1.4.71/maze.h:14:51: note: Function 'set_routes_to_net' argument 3 names different: declaration 'newnet' definition 'newflags'. qrouter-1.4.71/maze.c:764:47: note: Function 'set_routes_to_net' argument 3 names different: declaration 'newnet' definition 'newflags'. qrouter-1.4.71/maze.c:1185:50: style:inconclusive: Function 'ripup_net' argument 3 names different: declaration 'topmost' definition 'flagged'. [funcArgNamesDifferent] qrouter-1.4.71/maze.h:18:51: note: Function 'ripup_net' argument 3 names different: declaration 'topmost' definition 'flagged'. qrouter-1.4.71/maze.c:1185:50: note: Function 'ripup_net' argument 3 names different: declaration 'topmost' definition 'flagged'. qrouter-1.4.71/maze.c:292:12: style: Variable 'Pr' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/maze.c:1357:22: style: Parameter 'ept' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/maze.c:1364:18: style: Variable 'Pt' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/maze.c:2017:12: style: Variable 'Pr' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/maze.c:2087:21: style: Variable 'pri2' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/maze.c:1098:13: style: Variable 'netroutes' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/maze.c:1643:13: style: Variable 'layer' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/maze.c:2714:10: style: Variable 'lrend' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/maze.c:2791:15: style: Variable 'netobs1' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/maze.c:2792:15: style: Variable 'netobs2' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/maze.c:2507:21: style: Unused variable: pri2 [unusedVariable] qrouter-1.4.71/node.c:262:7: style:inconclusive: Boolean expression '((unsigned int)0x20000000)||((unsigned int)0x0000000f)' is used in bitwise operation. Did you mean '&&'? [bitwiseOnBoolean] qrouter-1.4.71/node.c:2927:14: error: Uninitialized struct member: dt.next [uninitStructMember] qrouter-1.4.71/node.c:2927:14: error: Uninitialized struct member: dt.layer [uninitStructMember] qrouter-1.4.71/node.c:819:7: style: Variable 'lay' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/node.c:936:11: style: Variable 'edist' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/node.c:864:20: style: Unused variable: deltaxy [unusedVariable] qrouter-1.4.71/node.c:1233:12: style: Unused variable: tpoint [unusedVariable] qrouter-1.4.71/node.c:1498:12: style: Unused variable: tpoint [unusedVariable] qrouter-1.4.71/node.c:2539:12: style: Unused variable: tpoint [unusedVariable] qrouter-1.4.71/node.c:2794:35: style: Unused variable: o [unusedVariable] qrouter-1.4.71/output.c:694:55: style:inconclusive: Boolean expression 'lnode->flags==0x40' is used in bitwise operation. Did you mean '||'? [bitwiseOnBoolean] qrouter-1.4.71/output.c:735:14: style:inconclusive: Boolean expression '((unsigned int)0x0000000f)!=0' is used in bitwise operation. Did you mean '&&'? [bitwiseOnBoolean] qrouter-1.4.71/output.c:694:55: style: Boolean result is used in bitwise operation. Clarify expression with parentheses. [clarifyCondition] qrouter-1.4.71/output.c:735:6: style: Suspicious condition (bitwise operator + comparison); Clarify expression with parentheses. [clarifyCondition] qrouter-1.4.71/output.c:735:14: style: Boolean result is used in bitwise operation. Clarify expression with parentheses. [clarifyCondition] qrouter-1.4.71/output.c:694:55: warning: Result of operator '|' is always true if one operand is non-zero. Did you intend to use '&'? [badBitmaskCheck] qrouter-1.4.71/output.c:90:22: style: Condition 'FailedNets!=(struct netlist_NETLIST)NULL' is always true [knownConditionTrueFalse] qrouter-1.4.71/output.c:2966:12: style: Condition '!net' is always true [knownConditionTrueFalse] qrouter-1.4.71/output.c:2905:14: note: Assignment 'net=NULL', assigned value is 0 qrouter-1.4.71/output.c:2966:12: note: Condition '!net' is always true qrouter-1.4.71/output.c:2925:10: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/output.c:3053:7: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf] qrouter-1.4.71/output.c:2866:2: error: Resource leak: fdef [resourceLeak] qrouter-1.4.71/output.c:2819:18: warning: Either the condition 'DEFfilename!=NULL' is redundant or there is possible null pointer dereference: DEFfilename. [nullPointerRedundantCheck] qrouter-1.4.71/output.c:2820:40: note: Assuming that condition 'DEFfilename!=NULL' is not redundant qrouter-1.4.71/output.c:2819:18: note: Null pointer dereference qrouter-1.4.71/output.c:1516:11: style: Variable 'lastseg' is reassigned a value before the old one has been used. [redundantAssignment] qrouter-1.4.71/output.c:1352:11: note: lastseg is assigned qrouter-1.4.71/output.c:1516:11: note: lastseg is overwritten qrouter-1.4.71/output.c:1760:23: style: Local variable 'dc' shadows outer variable [shadowVariable] qrouter-1.4.71/output.c:1324:11: note: Shadowed declaration qrouter-1.4.71/output.c:1760:23: note: Shadow variable qrouter-1.4.71/output.c:44:24: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:233:8: style: Parameter 'vianame' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:314:23: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:351:26: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:385:27: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:529:24: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:578:26: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:784:34: style: Parameter 'instname' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:812:18: style: Variable 'instname' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/output.c:812:29: style: Variable 'pinname' can be declared as pointer to const [constVariablePointer] qrouter-1.4.71/output.c:868:29: style: Parameter 'netname' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:2804:31: style: Parameter 'filename' can be declared as pointer to const [constParameterPointer] qrouter-1.4.71/output.c:786:9: style: Unused variable: net [unusedVariable] qrouter-1.4.71/output.c:808:9: style: Unused variable: net [unusedVariable] qrouter-1.4.71/output.c:1192:13: style: Unused variable: oval0 [unusedVariable] qrouter-1.4.71/output.c:3065:19: style: Variable 'lptr' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/output.c:2807:12: style: Unused variable: j [unusedVariable] qrouter-1.4.71/output.c:2811:10: style: Unused variable: node [unusedVariable] qrouter-1.4.71/output.c:2815:12: style: Variable 'need_cleanup' is assigned a value that is never used. [unreadVariable] qrouter-1.4.71/output.c:2902:14: style: Unused variable: instname [unusedVariable] qrouter-1.4.71/output.c:2902:25: style: Unused variable: pinname [unusedVariable] diff: head qrouter-1.4.71/graphics.c:777:47: style: Parameter 'interp' can be declared as pointer to const [constParameterPointer] head qrouter-1.4.71/graphics.c:778:11: style: Parameter 'objv' can be declared as const array [constParameter] head qrouter-1.4.71/main.c:65:39: style: Parameter 'lastlayer' can be declared as pointer to const [constParameterPointer] 2.13.0 qrouter-1.4.71/maze.c:1216:42: style: Condition 'rt->segments==NULL' is always false [knownConditionTrueFalse] head qrouter-1.4.71/maze.c:2764:10: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] head qrouter-1.4.71/maze.c:2764:10: portability: %d in format string (no. 2) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/maze.c:2764:10: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/maze.c:2764:10: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/output.c:90:22: style: Condition 'FailedNets!=(struct netlist_*)NULL' is always true [knownConditionTrueFalse] head qrouter-1.4.71/output.c:90:22: style: Condition 'FailedNets!=(struct netlist_NETLIST)NULL' is always true [knownConditionTrueFalse] 2.13.0 qrouter-1.4.71/qrouter.c:1035:22: style: Condition 'FailedNets!=(struct netlist_*)NULL' is always true [knownConditionTrueFalse] head qrouter-1.4.71/qrouter.c:1035:22: style: Condition 'FailedNets!=(struct netlist_NETLIST)NULL' is always true [knownConditionTrueFalse] head qrouter-1.4.71/qrouter.c:1354:6: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/qrouter.c:1354:6: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] head qrouter-1.4.71/qrouter.c:1508:6: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/qrouter.c:1508:6: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/qrouter.c:1528:22: style: Condition 'FailedNets!=(struct netlist_*)NULL' is always true [knownConditionTrueFalse] head qrouter-1.4.71/qrouter.c:1528:22: style: Condition 'FailedNets!=(struct netlist_NETLIST)NULL' is always true [knownConditionTrueFalse] head qrouter-1.4.71/qrouter.c:2105:9: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/qrouter.c:2105:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] head qrouter-1.4.71/qrouter.c:2278:5: portability: %d in format string (no. 1) requires 'int' but the argument type is 'u_int {aka unsigned int}'. [invalidPrintfArgType_sint] 2.13.0 qrouter-1.4.71/qrouter.c:2278:5: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] DONE