Automated Code Analysis

cppcheck analysis of gargoyle-free_2009-08-25-2.dsc

• ./tads/tads2/argize.c:43 [style] - Redundant assignment of "state" in switch
• ./tads/tads2/mcm.c:405 [error] - Possible null pointer dereference: o
• ./tads/tads2/mcm.c:406 [error] - Possible null pointer dereference: o
• ./tads/tads2/mcm.c:407 [error] - Possible null pointer dereference: o
• ./tads/tads2/mkchrtab.c:815 [error] - The given size 101 is mismatching
• ./tads/tads2/msdos/dosmktrx.c:1114 [error] - Resource leak: fpico
• ./tads/tads2/prscomp.c:623 [error] - Possible null pointer dereference: swctl
• ./tads/tads2/prscomp.c:218 [error] - Uninitialized variable: cur
• ./tads/tads2/prscomp.c:219 [error] - Uninitialized variable: cur
• ./tads/tads2/prscomp.c:1335 [error] - Uninitialized variable: saved_loc
• ./tads/tads2/tok.c:685 [error] - Possible null pointer dereference: child
• ./tads/tads2/tokth.c:154 [error] - Uninitialized variable: pg
• ./tads/tads3/test/data/define.c:37 [error] - mismatching number of '(' and ')' in this line: PASTE(1,
• ./tads/tads3/vmbiftio.cpp:1360 [error] - Uninitialized variable: msg
• ./tads/tads3/win32/test/capture.cpp:60 [error] - Buffer overrun possible for long cmd-line args
• ./terps/advsys/advexe.c:133 [error] - Null pointer dereference
• ./terps/advsys/advexe.c:136 [error] - Null pointer dereference
• ./terps/frotz/fastmem.c:375 [error] - Common realloc mistake: "zmp" nulled but not freed upon failure
• ./terps/jacl/jpp.c:197 [error] - Resource leak: inputFile
• ./terps/level9/Glk/glk.c:6698 [error] - Array 'argv[2]' index 2 out of bounds
• ./terps/magnetic/Generic/myth.c:94 [error] - Resource leak: file
• ./terps/magnetic/Generic/myth.c:102 [error] - Resource leak: file
• ./terps/magnetic/Generic/xtractpc.c:365 [error] - Buffer overrun possible for long cmd-line args
• ./terps/magnetic/Glk/glk.c:6274 [error] - Array 'argv[2]' index 2 out of bounds

---