Automated Code Analysis

cppcheck analysis of pennmush_1.8.2p8-1.dsc

• ./src/chunk.c:1521 [error] - Common realloc mistake: "regions" nulled but not freed upon failure
• ./src/conf.c:812 [error] - Undefined behaviour: loc is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./src/cque.c:263 [error] - Array 'env[10]' index 10 out of bounds
• ./src/extchat.c:2884 [error] - Undefined behaviour: buf is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./src/game.c:2429 [error] - Common realloc mistake: "errdblist" nulled but not freed upon failure
• ./src/game.c:2277 [error] - Mismatching allocation and deallocation: f
• ./src/game.c:832 [error] - Resource leak: f
• ./src/help.c:209 [error] - Resource leak: fp
• ./src/help.c:529 [error] - Resource leak: fp
• ./src/mysocket.c:1134 [error] - Memory leak: canon

---