Automated Code Analysis

cppcheck analysis of crossfire-client_1.50.0-2.dsc

• ./common/script.c:929 [error] - Possible null pointer dereference: data
• ./common/script.c:472 [error] - Common realloc mistake: "scripts" nulled but not freed upon failure
• ./common/script_lua.c:299 [error] - Common realloc mistake: "scripts" nulled but not freed upon failure
• ./gtk-v2/src/config.c:151 [error] - Dangerous usage of 'xml_basename' (strncpy doesn't always 0-terminate it)
• ./gtk-v2/src/config.c:1017 [error] - Dangerous usage of 'buf' (strncpy doesn't always 0-terminate it)
• ./gtk-v2/src/config.c:1097 [error] - Dangerous usage of 'buf' (strncpy doesn't always 0-terminate it)
• ./gtk-v2/src/opengl.c:160 [error] - Possible null pointer dereference: vi - otherwise it is redundant to check if vi is null at line 171
• ./gtk-v2/src/opengl.c:164 [error] - Possible null pointer dereference: vi - otherwise it is redundant to check if vi is null at line 171
• ./gtk-v2/src/opengl.c:166 [error] - Possible null pointer dereference: vi - otherwise it is redundant to check if vi is null at line 171
• ./gtk-v2/src/png.c:218 [error] - Memory leak: pixels
• ./gtk-v2/src/png.c:294 [error] - Data is allocated but not initialized: ndata
• ./sound-src/common.c:251 [error] - Resource leak: fp
• ./sound-src/common.c:295 [error] - Memory leak: path
• ./sound-src/common.c:339 [error] - Memory leak: path
• ./sound-src/cfsndserv.c:678 [error] - Resource leak: fp
• ./sound-src/cfsndserv.c:898 [error] - Memory leak: path
• ./sound-src/cfsndserv.c:942 [error] - Memory leak: path

---