Automated Code Analysis

cppcheck analysis of darktable_0.8-3.dsc

• ./src/LibRaw/dcraw/dcraw.c:8902 [error] - Unusual pointer arithmetic
• ./src/LibRaw/internal/dcraw_common.cpp:239 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:668 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:676 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:933 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:934 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:1236 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:2491 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:2497 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:2544 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:3011 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:3459 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:4304 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:6793 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:6998 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:6999 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:8832 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:8834 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:8856 [error] - Uninitialized variable: c
• ./src/LibRaw/internal/dcraw_common.cpp:3166 [error] - Uninitialized variable: c
• ./src/common/exif.cc:963 [error] - Dangerous usage of 'dtfilename' (strncpy doesn't always 0-terminate it)
• ./src/common/imageio_rawspeed.cc:121 [error] - Possible null pointer dereference: d - otherwise it is redundant to check if d is null at line 130
• ./src/common/imageio_rawspeed.cc:239 [error] - Possible null pointer dereference: d - otherwise it is redundant to check if d is null at line 248
• ./src/imageio/storage/flickr.c:144 [error] - Memory leak: _username
• ./src/imageio/storage/flickr.c:144 [error] - Memory leak: _user_token
• ./src/imageio/storage/flickr.c:792 [error] - Memory leak: d
• ./src/iop/bloom.c:199 [error] - Memory leak: scanline
• ./src/iop/lens.c:682 [error] - Possible null pointer dereference: cam - otherwise it is redundant to check if cam is null at line 685
• ./src/iop/watermark.c:157 [error] - Buffer access out-of-bounds: configdir
• ./src/iop/watermark.c:156 [error] - Buffer access out-of-bounds: datadir
• ./src/iop/watermark.c:453 [error] - Buffer access out-of-bounds: configdir
• ./src/iop/watermark.c:452 [error] - Buffer access out-of-bounds: datadir

---