Automated Code Analysis

cppcheck analysis of xpilot-ng_4.7.3-1.dsc

• ./contrib/maps2image/bdf2c.c:174 [error] - Resource leak: fl
• ./contrib/mapzip/mapzip.c:67 [error] - Resource leak: in
• ./src/client/clientrank.c:182 [possible error] - Resource leak: file
• ./src/client/clientrank.c:208 [possible error] - Resource leak: file
• ./src/client/gfx2d.c:221 [error] - Resource leak: f
• ./src/client/sdl/text.c:675 [error] - Uninitialized variable: text
• ./src/client/sdl/text.c:698 [error] - Uninitialized variable: text
• ./src/client/x11/welcome.c:324 [error] - Memory leak: server_names
• ./src/client/x11/welcome.c:324 [error] - Memory leak: server_addrs
• ./src/common/NT/winX11.c:344 [error] - Uninitialized variable: i
• ./src/mapedit/T_Handler.c:48 [error] - Undefined behaviour: returnval 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/mapedit/T_Handler.c:54 [error] - Undefined behaviour: returnval 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/mapedit/T_Handler.c:84 [error] - Undefined behaviour: returnval 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/mapedit/T_Handler.c:112 [error] - Undefined behaviour: returnval 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/mapedit/T_Handler.c:117 [error] - Undefined behaviour: returnval 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/mapedit/T_Handler.c:146 [error] - Undefined behaviour: returnval 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/mapedit/file.c:336 [error] - Memory leak: filename
• ./src/mapedit/file.c:408 [error] - Resource leak: fp
• ./src/mapedit/file.c:872 [error] - Undefined behaviour: returnval 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/mapedit/file.c:877 [error] - Undefined behaviour: returnval 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/mapedit/file.c:884 [error] - Undefined behaviour: returnval 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/mapedit/file.c:887 [error] - Undefined behaviour: returnval 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/mapedit/tools.c:898 [error] - Undefined behaviour: returnval 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/replay/xp-replay.c:3536 [possible error] - Resource leak: fp

---