[BlueZ v2 00/20] Fix a number of static analysis issues

Bastien Nocera
2024-05-10T12:14:02Z

Changes since v1:
- added 6 patches
- Fix syntax error in "client/gatt: Check write_value() retval"

Bastien Nocera (20):
adapter: Use false instead of 0 for bool
attrib/gatt: Guard against possible integer overflow
client/gatt: Don't pass negative fd on error
client/gatt: Check write_value() retval
client/main: Fix array access
client/main: Fix mismatched free
monitor/att: Fix memory leak
bap: Fix memory leaks
media: Fix memory leak
main: Fix memory leaks
isotest: Consider "0" fd to be valid
isotest: Fix error check after opening file
client/player: Fix copy/paste error
shared/vcp: Fix copy/paste error
isotest: Fix fd leak
iso-tester: Fix fd leak
sdp: Fix use of uninitialised memory
monitor: Work-around memory leak warning
avrcp: Fix uninitialised memory usage
main: Simplify variable assignment

attrib/gatt.c | 8 ++++---
client/gatt.c | 21 +++++++++++++++----
client/main.c | 7 ++++++-
client/player.c | 2 +-
lib/sdp.c | 2 +-
monitor/att.c | 19 +++++++++++++++++
monitor/jlink.c | 3 ++-
profiles/audio/avrcp.c | 10 ++++-----
profiles/audio/bap.c | 47 +++++++++++++++++++++++++++++-------------
profiles/audio/media.c | 1 +
src/adapter.c | 2 +-
src/main.c | 16 +++++++-------
src/shared/vcp.c | 2 +-
tools/iso-tester.c | 1 +
tools/isotest.c | 6 ++++--
15 files changed, 104 insertions(+), 43 deletions(-)

--
2.44.0