Automated Code Analysis

cppcheck analysis of sane-backends_1.0.19-26.dsc

• ./backend/artec_eplus48u.c:3717 [error] - Resource leak: f
• ./backend/canon630u-common.c:576 [error] - Memory leak: buf
• ./backend/canon_pp-dev.c:516 [error] - Resource leak: fd
• ./backend/dc240.c:1434 [error] - Buffer access out-of-bounds: path
• ./backend/dc240.c:1437 [error] - Buffer access out-of-bounds: path
• ./backend/dc240.c:1674 [error] - Buffer access out-of-bounds: path
• ./backend/dc240.c:1951 [error] - Buffer access out-of-bounds: path
• ./backend/dc240.c:1954 [error] - Buffer access out-of-bounds: path
• ./backend/dc25.c:1272 [error] - Memory leak: horizontal_interpolation
• ./backend/dc25.c:1303 [error] - Memory leak: red
• ./backend/dc25.c:1303 [error] - Memory leak: green
• ./backend/dll.c:971 [error] - Common realloc mistake: "devlist" nulled but not freed upon failure
• ./backend/dll.c:1030 [error] - Common realloc mistake: "devlist" nulled but not freed upon failure
• ./backend/epson.c:1876 [error] - Memory leak: head
• ./backend/epson.c:5962 [error] - Memory leak: ident
• ./backend/genesys.c:4875 [error] - Possible null pointer dereference: dev - otherwise it is redundant to check if dev is null at line 4876
• ./backend/genesys.c:4911 [error] - Common realloc mistake: "new_dev" nulled but not freed upon failure
• ./backend/genesys.c:1901 [error] - Memory leak: calibration_data
• ./backend/genesys.c:1901 [error] - Memory leak: all_data
• ./backend/genesys.c:2067 [error] - Memory leak: all_data_8
• ./backend/genesys.c:3499 [error] - Memory leak: first_line
• ./backend/genesys_gl646.c:3539 [error] - Memory leak: first_line
• ./backend/genesys_gl646.c:3539 [error] - Memory leak: second_line
• ./backend/genesys_gl646.c:3762 [error] - Memory leak: line
• ./backend/genesys_gl841.c:4872 [error] - Memory leak: line
• ./backend/genesys_gl841.c:5216 [error] - Memory leak: line
• ./backend/gphoto2.c:1597 [error] - Buffer access out-of-bounds: path
• ./backend/gphoto2.c:1768 [error] - Buffer access out-of-bounds: path
• ./backend/gt68xx.c:846 [error] - Common realloc mistake: "new_dev" nulled but not freed upon failure
• ./backend/gt68xx.c:894 [error] - Dangerous usage of 'dirname' (strncpy doesn't always 0-terminate it)
• ./backend/hp3900_rts8822.c:837 [error] - Common realloc mistake: "rst" nulled but not freed upon failure
• ./backend/hp3900_rts8822.c:3605 [error] - Memory leak: resource
• ./backend/hp3900_rts8822.c:14926 [error] - Memory leak: myscancfg
• ./backend/hp3900_rts8822.c:14926 [error] - Memory leak: avg_colors
• ./backend/hp3900_rts8822.c:14997 [error] - Memory leak: myscancfg
• ./backend/hp3900_rts8822.c:15195 [error] - Memory leak: myscancfg
• ./backend/hp3900_rts8822.c:15265 [error] - Memory leak: myscancfg
• ./backend/hp5400_internal.c:342 [error] - Memory leak: buffer
• ./backend/hp5400_internal.c:719 [error] - Resource leak: file
• ./backend/hp5590.c:517 [error] - Memory leak: opts
• ./backend/hs2p-scsi.c:1224 [error] - Buffer access out-of-bounds: s.gamma_table
• ./backend/lexmark_low.c:2025 [error] - Resource leak: img
• ./backend/lexmark_low.c:2559 [error] - Memory leak: buffer
• ./backend/ma1509.c:1865 [error] - Possible null pointer dereference: s - otherwise it is redundant to check if s is null at line 1868
• ./backend/ma1509.c:681 [error] - Common realloc mistake: "new_dev" nulled but not freed upon failure
• ./backend/mustek.c:5152 [error] - Common realloc mistake: "new_dev" nulled but not freed upon failure
• ./backend/mustek.c:2234 [error] - Memory leak: cmd2
• ./backend/mustek.c:4988 [error] - Memory leak: extra
• ./backend/mustek_pp.c:275 [error] - Memory leak: buffer
• ./backend/mustek_pp.c:1159 [error] - Memory leak: hndl
• ./backend/mustek_usb.c:577 [error] - Common realloc mistake: "new_dev" nulled but not freed upon failure
• ./backend/mustek_usb_high.c:297 [error] - Memory leak: buffer
• ./backend/mustek_usb_high.c:1367 [error] - Memory leak: buffer
• ./backend/mustek_usb2_asic.c:2518 [error] - Memory leak: temps
• ./backend/mustek_usb2_asic.c:3874 [error] - Memory leak: lpMotorStepsTable
• ./backend/mustek_usb2_asic.c:4898 [error] - Memory leak: lpMotorStepsTable
• ./backend/mustek_usb2_asic.c:4898 [error] - Memory leak: lpMotorTable
• ./backend/mustek_usb2_reflective.c:659 [error] - Memory leak: lpCalData
• ./backend/mustek_usb2_reflective.c:1201 [error] - Memory leak: lpCalData
• ./backend/mustek_usb2_reflective.c:1201 [error] - Resource leak: stream
• ./backend/mustek_usb2_reflective.c:1520 [error] - Memory leak: lpWhiteData
• ./backend/mustek_usb2_reflective.c:1520 [error] - Memory leak: lpDarkData
• ./backend/mustek_usb2_transparent.c:696 [error] - Memory leak: lpCalData
• ./backend/mustek_usb2_transparent.c:1190 [error] - Memory leak: lpCalData
• ./backend/mustek_usb2_transparent.c:1374 [error] - Memory leak: lpWhiteData
• ./backend/mustek_usb2_transparent.c:1374 [error] - Memory leak: lpDarkData
• ./backend/pint.c:134 [error] - Resource leak: fd
• ./backend/pixma_common.c:486 [error] - Possible null pointer dereference: s - otherwise it is redundant to check if s is null at line 487
• ./backend/sm3600-scanutil.c:386 [error] - Memory leak: achBuf
• ./backend/sm3840.c:801 [error] - Possible null pointer dereference: dev - otherwise it is redundant to check if dev is null at line 802
• ./backend/teco2.c:1520 [error] - Memory leak: tmp_min_buf
• ./backend/teco2.c:1520 [error] - Memory leak: tmp_max_buf
• ./backend/test.c:1417 [error] - Buffer access out-of-bounds: caps
• ./backend/test.c:1419 [error] - Buffer access out-of-bounds: caps
• ./backend/test.c:1421 [error] - Buffer access out-of-bounds: caps
• ./backend/test.c:1423 [error] - Buffer access out-of-bounds: caps
• ./backend/test.c:1425 [error] - Buffer access out-of-bounds: caps
• ./backend/test.c:1427 [error] - Buffer access out-of-bounds: caps
• ./backend/test.c:1429 [error] - Buffer access out-of-bounds: caps
• ./backend/umax.c:5908 [error] - Array 'scan_mode_list[7]' index 4294967295 out of bounds
• ./backend/umax.c:2585 [error] - Memory leak: average
• ./backend/umax1220u-common.c:1185 [error] - Memory leak: img
• ./backend/umax1220u-common.c:1243 [error] - Memory leak: img
• ./backend/umax1220u-common.c:1322 [error] - Memory leak: p
• ./backend/umax1220u-common.c:1486 [error] - Memory leak: p
• ./backend/umax1220u-common.c:987 [error] - Array 'buf[512]' index 521 out of bounds
• ./backend/umax_pp_low.c:11107 [error] - Memory leak: buffer
• ./backend/umax_pp_low.c:11117 [error] - Memory leak: dest
• ./backend/umax_pp_low.c:12928 [error] - Memory leak: data
• ./backend/umax_pp_low.c:13162 [error] - Memory leak: commit
• ./backend/umax_pp_low.c:13162 [error] - Memory leak: data
• ./backend/umax_pp_low.c:1259 [error] - Resource leak: fd
• ./backend/v4l.c:559 [error] - Memory leak: s
• ./frontend/saned.c:435 [error] - Common realloc mistake: "handle" nulled but not freed upon failure
• ./sanei/sanei_DomainOS.c:214 [error] - Common realloc mistake: "DomainFdInfo" nulled but not freed upon failure
• ./sanei/sanei_codec_ascii.c:228 [error] - Common realloc mistake: "str" nulled but not freed upon failure
• ./sanei/sanei_scsi.c:1455 [error] - Common realloc mistake: "fd_info" nulled but not freed upon failure
• ./tools/sane-desc.c:1427 [error] - Resource leak: dir

---