Automated Code Analysis

cppcheck analysis of voxbo_1.8.5~svn1241-1.dsc

• ./crunch/norm.cpp:364 [error] - Uninitialized variable: tmp
• ./crunch/norm_brainwarp.cpp:628 [error] - Mismatching allocation and deallocation: alpha
• ./crunch/norm_brainwarp.cpp:629 [error] - Mismatching allocation and deallocation: beta
• ./crunch/norm_brainwarp.cpp:630 [error] - Mismatching allocation and deallocation: var
• ./db_import/add_aal_name.cpp:68 [error] - Resource leak: ifp
• ./db_import/add_aal_relation.cpp:74 [error] - Resource leak: ifp
• ./db_import/readBrodmann.cpp:76 [error] - Resource leak: ifp
• ./db_import/readMarianna.cpp:85 [error] - Resource leak: ifp
• ./getput/getdata.cpp:512 [error] - Memory leak: data
• ./getput/getdata.cpp:720 [error] - Resource leak: f
• ./getput/getdata.cpp:731 [error] - Resource leak: f
• ./getput/getdata.cpp:733 [error] - Resource leak: f
• ./getput/getdata.cpp:770 [error] - Resource leak: f
• ./getput/getdata.cpp:793 [error] - Resource leak: f
• ./getput/getdata.cpp:814 [error] - Resource leak: f
• ./getput/getdata.cpp:819 [error] - Resource leak: f
• ./getput/getdata.cpp:823 [error] - Resource leak: f
• ./getput/getdata.cpp:826 [error] - Resource leak: f
• ./getput/getdata.cpp:653 [error] - Dereferencing 'c' after it is deallocated / released
• ./getput/putdata.cpp:390 [error] - Resource leak: f
• ./getput/putdata.cpp:549 [error] - Memory leak: chunk
• ./lib/dicom.cpp:105 [error] - Resource leak: ifile
• ./lib/dicom.cpp:1063 [error] - Resource leak: ifile
• ./lib/dicom.cpp:1849 [error] - Resource leak: ifile
• ./lib/dicom.cpp:610 [error] - Invalid radix in call to strtol or strtoul. Must be 0 or 2-36
• ./lib/ff_da.cpp:136 [error] - Resource leak: fp
• ./lib/makestatcub.cpp:627 [error] - Possible null pointer dereference: Betas - otherwise it is redundant to check if Betas is null at line 658
• ./lib/makestatcub.cpp:628 [error] - Possible null pointer dereference: isoBetas - otherwise it is redundant to check if isoBetas is null at line 656
• ./lib/makestatcub.cpp:629 [error] - Possible null pointer dereference: TisoBetasInvVarIsoBetas - otherwise it is redundant to check if TisoBetasInvVarIsoBetas is null at line 657
• ./lib/makestatcub.cpp:629 [error] - Possible null pointer dereference: isoBetas - otherwise it is redundant to check if isoBetas is null at line 656
• ./lib/statthreshold.cpp:619 [error] - Possible null pointer dereference: pval_rf
• ./lib/statthreshold.cpp:630 [error] - Possible null pointer dereference: pval_bon
• ./lib/statthreshold.cpp:679 [error] - Possible null pointer dereference: rhoRow
• ./lib/statthreshold.cpp:688 [error] - Possible null pointer dereference: pval1
• ./lib/statthreshold.cpp:986 [error] - Memory leak: prodfft
• ./lib/vbexecdef.cpp:90 [error] - Dangerous iterator usage. After erase the iterator is invalid so dereferencing it or comparing it with another iterator is invalid.
• ./scheduler/voxbo.cpp:1113 [error] - Resource leak: numfile
• ./stand_alone/perm.cpp:132 [error] - Mismatching allocation and deallocation: signPerm
• ./stand_alone/perm.cpp:248 [error] - Mismatching allocation and deallocation: signPerm
• ./stand_alone/sliceacq.cpp:710 [error] - Invalid radix in call to strtol or strtoul. Must be 0 or 2-36
• ./stand_alone/sliceacq.cpp:286 [error] - Division by zero
• ./stand_alone/tesjoin.cpp:746 [error] - Resource leak: pFile
• ./vbsequence/mainwindow.cpp:630 [error] - Possible null pointer dereference: action - otherwise it is redundant to check if action is null at line 632
• ./vbsequence/standalone_demo.cpp:28 [error] - Invalid number of character (() when these macros are defined: ''.
• ./vbsequence/vbsequenceview.cpp:248 [error] - Possible null pointer dereference: action - otherwise it is redundant to check if action is null at line 250

---