Automated Code Analysis

cppcheck analysis of scummvm_1.0.0-3.dsc

• ./backends/platform/dc/dc.h:193 [possible error] - Memory leak: OSystem_Dreamcast::_timer
• ./backends/platform/dc/dc.h:208 [possible error] - Memory leak: OSystem_Dreamcast::screen
• ./backends/platform/dc/dc.h:210 [possible error] - Memory leak: OSystem_Dreamcast::overlay
• ./backends/platform/dc/selector.cpp:454 [error] - Mismatching allocation and deallocation: games
• ./backends/platform/ds/arm9/source/ramsave.cpp:95 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/ds/arm9/source/ramsave.cpp:101 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/ds/arm9/source/ramsave.cpp:122 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/ds/arm9/source/ramsave.cpp:270 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/ds/arm9/source/ramsave.cpp:285 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/ds/arm9/source/ramsave.cpp:288 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/ds/arm9/source/ramsave.cpp:299 [possible error] - Mismatching allocation and deallocation: DSSaveFile::saveData
• ./backends/platform/gp2x/events.cpp:603 [possible error] - Memory leak: act
• ./backends/platform/gp2x/gp2x.cpp:240 [possible error] - Resource leak: newfp
• ./backends/platform/gp2x/gp2x.cpp:161 [possible error] - Resource leak: newfp
• ./backends/platform/gp2xwiz/gp2xwiz-main.cpp:168 [possible error] - Resource leak: newfp
• ./backends/platform/gp2xwiz/gp2xwiz-main.cpp:128 [possible error] - Resource leak: newfp
• ./backends/platform/iphone/osys_main.cpp:296 [error] - Resource leak: newfp
• ./backends/platform/iphone/osys_main.cpp:229 [error] - Dangerous usage of 'buf' (strncpy doesn't always 0-terminate it)
• ./backends/platform/iphone/osys_main.cpp:241 [error] - Dangerous usage of 'buf' (strncpy doesn't always 0-terminate it)
• ./backends/platform/ps2/Gs2dScreen.h:99 [possible error] - Memory leak: Gs2dScreen::_dmaPipe
• ./backends/platform/ps2/Gs2dScreen.h:144 [possible error] - Memory leak: Gs2dScreen::_animStack
• ./backends/platform/wince/CEException.cpp:79 [error] - Buffer access out-of-bounds
• ./backends/platform/wince/wince-sdl.cpp:788 [possible error] - Memory leak: keyboard
• ./backends/platform/wince/wince-sdl.cpp:1081 [possible error] - Memory leak: panel
• ./base/main.cpp:276 [possible error] - Memory leak: act
• ./engines/agi/preagi_mickey.cpp:224 [error] - Memory leak: buffer
• ./engines/agi/preagi_mickey.cpp:247 [error] - Memory leak: buffer
• ./engines/agi/preagi_mickey.cpp:734 [error] - Memory leak: buffer
• ./engines/agi/preagi_mickey.cpp:756 [error] - Memory leak: buffer
• ./engines/agi/preagi_mickey.cpp:935 [error] - Memory leak: buffer
• ./engines/agi/preagi_winnie.cpp:1065 [error] - Memory leak: buffer
• ./engines/agos/sound.cpp:496 [possible error] - Memory leak: file
• ./engines/agos/sound.cpp:548 [possible error] - Memory leak: file
• ./engines/agos/sound.cpp:871 [possible error] - Memory leak: file
• ./engines/agos/string.cpp:317 [error] - Undefined behaviour: filename 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.
• ./engines/agos/subroutine.cpp:358 [error] - Undefined behaviour: filename 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.
• ./engines/dialogs.cpp:82 [possible error] - Memory leak: title
• ./engines/dialogs.cpp:117 [possible error] - Memory leak: version
• ./engines/dialogs.cpp:117 [possible error] - Memory leak: title
• ./engines/gob/save/saveconverter.cpp:277 [possible error] - Memory leak: sprite
• ./engines/gob/save/saveconverter_v2.cpp:70 [error] - Mismatching allocation and deallocation: desc
• ./engines/gob/save/saveconverter_v3.cpp:78 [error] - Mismatching allocation and deallocation: desc
• ./engines/gob/save/saveconverter_v4.cpp:74 [error] - Mismatching allocation and deallocation: desc
• ./engines/gob/save/saveconverter_v6.cpp:70 [error] - Mismatching allocation and deallocation: desc
• ./engines/gob/sound/sound.cpp:697 [possible error] - Memory leak: fileName
• ./engines/groovie/cell.cpp:779 [error] - Uninitialized variable: result
• ./engines/kyra/script_lol.cpp:2145 [possible error] - Memory leak: mov
• ./engines/kyra/sound_towns.cpp:159 [possible error] - Memory leak: Towns_EuphonyTrackQueue::_next
• ./engines/kyra/text_lok.cpp:345 [error] - Dangerous usage of 'sentenceCommand' (strncpy doesn't always 0-terminate it)
• ./engines/lure/intro.cpp:135 [possible error] - Memory leak: anim
• ./engines/lure/lure.cpp:136 [possible error] - Memory leak: gameInstance
• ./engines/lure/menu.cpp:640 [possible error] - Memory leak: s
• ./engines/m4/console.cpp:220 [possible error] - Memory leak: asset
• ./engines/m4/graphics.cpp:103 [error] - Memory leak: walkMap
• ./engines/m4/resource.cpp:155 [error] - Undefined behaviour: destName 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.
• ./engines/m4/scene.cpp:313 [possible error] - Memory leak: _sceneSpriteCodes
• ./engines/saga/objectmap.cpp:195 [possible error] - Memory leak: points
• ./engines/saga/resource_hrs.cpp:66 [error] - Mismatching allocation and deallocation: origin
• ./engines/saga/resource_hrs.cpp:80 [error] - Mismatching allocation and deallocation: origin
• ./engines/saga/resource_hrs.cpp:102 [error] - Mismatching allocation and deallocation: origin
• ./engines/sci/decompressor.cpp:563 [error] - Memory leak: cc_lengths
• ./engines/sci/engine/klists.cpp:444 [error] - Memory leak: temp_array
• ./engines/sci/engine/kmenu.cpp:83 [error] - Memory leak: tmp
• ./engines/sci/gfx/gfx_gui.cpp:225 [possible error] - Memory leak: win
• ./engines/sci/gfx/res_pal.cpp:126 [possible error] - Memory leak: retval
• ./engines/sci/gfx/seq_decoder.cpp:185 [error] - Memory leak: buf
• ./engines/sci/sfx/softseq/amiga.cpp:417 [error] - Memory leak: instrument
• ./engines/sci/sfx/softseq/amiga.cpp:591 [error] - Memory leak: buffers
• ./engines/sci/sfx/softseq/amiga.cpp:417 [error] - Memory leak: instrument.samples
• ./engines/scumm/camera.cpp:298 [error] - Possible null pointer dereference: a
• ./engines/scumm/script.cpp:145 [possible error] - Buffer access out-of-bounds
• ./engines/scumm/sound.cpp:2061 [error] - Memory leak: src_ptr
• ./engines/sword1/control.cpp:1270 [error] - Mismatching allocation and deallocation: saveData
• ./engines/sword1/control.cpp:1301 [error] - Mismatching allocation and deallocation: saveData
• ./engines/sword2/render.cpp:600 [error] - Memory leak: tileChunk
• ./engines/sword2/resman.cpp:182 [error] - Mismatching allocation and deallocation: cdInf
• ./engines/sword2/sprite.cpp:542 [error] - Memory leak: tempBuf2
• ./engines/tinsel/tinlib.cpp:3820 [error] - Uninitialized variable: time
• ./graphics/video/smk_decoder.cpp:234 [possible error] - Memory leak: BigHuffmanTree::_loBytes
• ./graphics/video/smk_decoder.cpp:235 [possible error] - Memory leak: BigHuffmanTree::_hiBytes
• ./gui/GuiManager.cpp:98 [possible error] - Memory leak: act
• ./gui/launcher.cpp:131 [possible error] - Memory leak: EditGameDialog::_descriptionWidget
• ./gui/launcher.cpp:132 [possible error] - Memory leak: EditGameDialog::_domainWidget
• ./gui/launcher.cpp:134 [possible error] - Memory leak: EditGameDialog::_gamePathWidget
• ./gui/launcher.cpp:135 [possible error] - Memory leak: EditGameDialog::_extraPathWidget
• ./gui/launcher.cpp:136 [possible error] - Memory leak: EditGameDialog::_savePathWidget
• ./gui/launcher.cpp:138 [possible error] - Memory leak: EditGameDialog::_langPopUpDesc
• ./gui/launcher.cpp:139 [possible error] - Memory leak: EditGameDialog::_langPopUp
• ./gui/launcher.cpp:140 [possible error] - Memory leak: EditGameDialog::_platformPopUpDesc
• ./gui/launcher.cpp:141 [possible error] - Memory leak: EditGameDialog::_platformPopUp
• ./gui/launcher.cpp:143 [possible error] - Memory leak: EditGameDialog::_globalGraphicsOverride
• ./gui/launcher.cpp:144 [possible error] - Memory leak: EditGameDialog::_globalAudioOverride
• ./gui/launcher.cpp:145 [possible error] - Memory leak: EditGameDialog::_globalMIDIOverride
• ./gui/launcher.cpp:146 [possible error] - Memory leak: EditGameDialog::_globalVolumeOverride
• ./tools/convbdf.c:940 [error] - Resource leak: ofp
• ./tools/create_lure/create_lure_dat.cpp:889 [error] - Mismatching allocation and deallocation: includeAnim
• ./tools/create_lure/process_actions.cpp:548 [error] - Mismatching allocation and deallocation: randomActions
• ./tools/md5table.c:372 [error] - Memory leak: generationDate
• ./tools/sci/scipack.cpp:144 [error] - Resource leak: resource_000
• ./tools/sci/sciunpack.cpp:440 [error] - Resource leak: outf
• ./tools/skycpt/cptcompiler.cpp:527 [error] - Memory leak: buf288

---