Automated Code Analysis

cppcheck analysis of freedink_1.08.20090120-1.dsc

• ./contrib/bmpinspect.c:155 [error] - Resource leak: file
• ./contrib/bmpinspect.c:155 [error] - Memory leak: palette
• ./contrib/dinkdat_inspect.c:51 [error] - Resource leak: f
• ./contrib/ffinspect.c:90 [error] - Resource leak: file
• ./gnulib/lib/getcwd.c:428 [possible error] - Resource leak: fd
• ./src/dinkc_bindings.c:781 [error] - Buffer access out-of-bounds: play.button
• ./src/freedinkedit.c:4625 [error] - Undefined behaviour: msg 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/freedinkedit.c:4925 [error] - Undefined behaviour: in_temp 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/gfx_fonts.c:273 [possible error] - Memory leak: filename
• ./src/input.c:55 [error] - Buffer access out-of-bounds: play.button
• ./src/update_frame.c:961 [error] - Undefined behaviour: msg 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.

---