Automated Code Analysis

cppcheck analysis of blender_2.49.2~dfsg-2.dsc

• ./extern/binreloc/binreloc.c:250 [error] - Common realloc mistake: "address_string" nulled but not freed upon failure
• ./extern/ode/dist/contrib/dRay/dxRay.h:9 [error] - Array 'Rot[3]' index 3 out of bounds
• ./extern/ode/dist/drawstuff/src/windows.cpp:112 [error] - Array 'keybuffer[16]' index 16 out of bounds
• ./extern/qhull/src/global.c:106 [error] - Array 'chkopt[2]' index 2 out of bounds
• ./extern/qhull/src/global.c:117 [error] - Array 'chkopt[2]' index 2 out of bounds
• ./extern/qhull/src/global.c:110 [error] - Array 'chkopt2[2]' index 2 out of bounds
• ./extern/qhull/src/global.c:111 [error] - Array 'chkopt2[2]' index 3 out of bounds
• ./extern/qhull/src/mem.c:129 [error] - Data is allocated but not initialized: newbuffer
• ./extern/qhull/src/merge.c:235 [error] - Possible null pointer dereference: merge - otherwise it is redundant to check if merge is null at line 238
• ./extern/qhull/src/merge.c:236 [error] - Possible null pointer dereference: merge - otherwise it is redundant to check if merge is null at line 238
• ./extern/qhull/src/merge.c:237 [error] - Possible null pointer dereference: merge - otherwise it is redundant to check if merge is null at line 238
• ./extern/qhull/src/poly2.c:1079 [error] - Possible null pointer dereference: ridge - otherwise it is redundant to check if ridge is null at line 1082
• ./extern/qhull/src/poly2.c:1080 [error] - Possible null pointer dereference: ridge - otherwise it is redundant to check if ridge is null at line 1082
• ./intern/SoundSystem/openal/SND_OpenALDevice.cpp:146 [error] - Resource leak: fp
• ./intern/bsp/test/BSP_GhostTest/plyfile.c:256 [error] - Memory leak: name
• ./intern/bsp/test/BSP_GhostTest/plyfile.c:752 [error] - Memory leak: plyfile
• ./intern/bsp/test/BSP_GhostTest/plyfile.c:866 [error] - Memory leak: name
• ./intern/decimation/test/decimate_glut_test/intern/ply/plyfile.c:256 [error] - Memory leak: name
• ./intern/decimation/test/decimate_glut_test/intern/ply/plyfile.c:752 [error] - Memory leak: plyfile
• ./intern/decimation/test/decimate_glut_test/intern/ply/plyfile.c:866 [error] - Memory leak: name
• ./intern/guardedalloc/intern/mmap_win.c:145 [error] - Possible null pointer dereference: mm
• ./intern/guardedalloc/intern/mmap_win.c:146 [error] - Possible null pointer dereference: mm
• ./intern/guardedalloc/intern/mmap_win.c:147 [error] - Possible null pointer dereference: mm
• ./release/plugins/sequence/color-correction-hsv.c:190 [error] - Possible null pointer dereference: ibuf1 - otherwise it is redundant to check if ibuf1 is null at line 192
• ./release/plugins/sequence/color-correction-yuv.c:123 [error] - Possible null pointer dereference: ibuf1 - otherwise it is redundant to check if ibuf1 is null at line 125
• ./source/blender/blenkernel/intern/DerivedMesh.c:2394 [error] - syntax error
• ./source/blender/blenkernel/intern/exotic.c:400 [error] - Common realloc mistake: "vertdata" nulled but not freed upon failure
• ./source/blender/blenkernel/intern/lattice.c:877 [error] - syntax error
• ./source/blender/blenkernel/intern/mesh.c:1060 [error] - syntax error
• ./source/blender/blenkernel/intern/multires.c:876 [error] - Possible null pointer dereference: efa
• ./source/blender/blenkernel/intern/writeffmpeg.c:367 [error] - Dangerous usage of 'name' (strncpy doesn't always 0-terminate it)
• ./source/blender/blenlib/intern/scanfill.c:428 [error] - Possible null pointer dereference: ed1
• ./source/blender/blenlib/intern/storage.c:297 [error] - Common realloc mistake: "files" nulled but not freed upon failure
• ./source/blender/blenlib/intern/storage.c:485 [error] - Resource leak: file
• ./source/blender/blenlib/intern/util.c:412 [error] - Possible null pointer dereference: previous - otherwise it is redundant to check if previous is null at line 419
• ./source/blender/imbuf/intern/IMB_imginfo.h:34 [error] - Invalid number of character ({) when these macros are defined: '__cplusplus'.
• ./source/blender/imbuf/intern/readimage.c:315 [error] - Resource leak: file
• ./source/blender/imbuf/intern/targa.c:299 [error] - Resource leak: fildes
• ./source/blender/python/BPY_menus.c:521 [error] - Resource leak: fp
• ./source/blender/python/api2_2x/Blender.c:640 [error] - Resource leak: file
• ./source/blender/python/api2_2x/Mesh.c:2525 [error] - Possible null pointer dereference: edge - otherwise it is redundant to check if edge is null at line 2527
• ./source/blender/python/api2_2x/Text3d.c:1247 [error] - Resource leak: file
• ./source/blender/python/api2_2x/Window.c:959 [error] - Array 'undo_str[12]' index 63 out of bounds
• ./source/blender/radiosity/intern/source/radpostprocess.c:467 [error] - Possible null pointer dereference: face
• ./source/blender/src/drawview.c:1298 [error] - Undefined behaviour: shapes 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.
• ./source/blender/src/editipo.c:2265 [error] - Uninitialized variable: icu
• ./source/blender/src/editmesh_tools.c:5994 [error] - Undefined behaviour: menu 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.
• ./source/blender/src/editseq.c:1092 [error] - Dangerous usage of 'str' (strncpy doesn't always 0-terminate it)
• ./source/blender/src/editseq.c:1094 [error] - Dangerous usage of 'str' (strncpy doesn't always 0-terminate it)
• ./source/blender/src/editseq.c:1185 [error] - Dangerous usage of 'str' (strncpy doesn't always 0-terminate it)
• ./source/blender/src/editseq.c:1253 [error] - Dangerous usage of 'str' (strncpy doesn't always 0-terminate it)
• ./source/blender/src/editseq.c:1255 [error] - Dangerous usage of 'str' (strncpy doesn't always 0-terminate it)
• ./source/blender/src/header_script.c:165 [error] - Possible null pointer dereference: sc - otherwise it is redundant to check if sc is null at line 168
• ./source/gameengine/Converter/KX_IpoConvert.cpp:379 [error] - Memory leak: ipoList
• ./source/gameengine/Converter/KX_IpoConvert.cpp:445 [error] - Memory leak: ipoList
• ./source/gameengine/Converter/KX_IpoConvert.cpp:495 [error] - Memory leak: ipoList
• ./source/gameengine/Converter/KX_IpoConvert.cpp:563 [error] - Memory leak: ipoList
• ./source/gameengine/Converter/KX_IpoConvert.cpp:754 [error] - Memory leak: ipoList
• ./source/gameengine/GameLogic/SCA_TimeEventManager.cpp:111 [error] - Same iterator is used with both m_timevalues and m_timevalues
• ./source/gameengine/GamePlayer/ghost/GPG_Application.cpp:568 [error] - Possible null pointer dereference: m_rasterizer - otherwise it is redundant to check if m_rasterizer is null at line 569
• ./source/gameengine/Ketsji/KX_ConvertPhysicsObjects.cpp:295 [error] - Memory leak: smmaterial
• ./source/gameengine/Ketsji/KX_WorldIpoController.cpp:55 [error] - Possible null pointer dereference: world
• ./source/gameengine/Ketsji/KX_WorldIpoController.cpp:59 [error] - Possible null pointer dereference: world
• ./source/gameengine/Ketsji/KX_WorldIpoController.cpp:60 [error] - Possible null pointer dereference: world
• ./source/gameengine/Ketsji/KX_WorldIpoController.cpp:61 [error] - Possible null pointer dereference: world
• ./source/gameengine/Ketsji/KX_WorldIpoController.cpp:65 [error] - Possible null pointer dereference: world
• ./source/gameengine/Physics/Bullet/CcdPhysicsEnvironment.cpp:1531 [error] - Uninitialized variable: o

---