mbox series

[BlueZ,0/9] Fix a number of static analysis issues #3

Message ID 20240530150057.444585-1-hadess@hadess.net (mailing list archive)
Headers show
Series Fix a number of static analysis issues #3 | expand

Message

Bastien Nocera May 30, 2024, 2:57 p.m. UTC 
14 defects fixed, and 1 error check added.

Let me know whether there's any problems with the implementation, I'm
thinking in particular of the avdtp changes which are pretty invasive.

Cheers

Bastien Nocera (9):
  rctest: Fix possible overrun
  mgmt-tester: Fix buffer overrun
  l2test: Add missing error checking
  rfkill: Avoid using a signed int for an unsigned variable
  shared/mainloop: Fix integer overflow
  sdp: Fix ineffective error guard
  obexd: Fix buffer overrun
  bap: Fix more memory leaks on error
  avdtp: Fix manipulating struct as an array

 gobex/gobex.c                |  2 +-
 lib/sdp.c                    |  8 ++++----
 profiles/audio/avdtp.c       | 33 ++++++++++++++++++++++-----------
 profiles/audio/bap.c         |  5 ++++-
 src/rfkill.c                 |  2 +-
 src/shared/mainloop-notify.c |  3 ++-
 tools/l2test.c               |  5 +++++
 tools/mgmt-tester.c          |  2 +-
 tools/rctest.c               |  3 ++-
 9 files changed, 42 insertions(+), 21 deletions(-)

Comments

patchwork-bot+bluetooth@kernel.org June 3, 2024, 7:20 p.m. UTC | #1 
Hello:

This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Thu, 30 May 2024 16:57:54 +0200 you wrote:
> 14 defects fixed, and 1 error check added.
> 
> Let me know whether there's any problems with the implementation, I'm
> thinking in particular of the avdtp changes which are pretty invasive.
> 
> Cheers
> 
> [...]

Here is the summary with links:
  - [BlueZ,1/9] rctest: Fix possible overrun
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=24cf04939502
  - [BlueZ,2/9] mgmt-tester: Fix buffer overrun
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=aa54087f13d5
  - [BlueZ,3/9] l2test: Add missing error checking
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=ccec5e8ef171
  - [BlueZ,4/9] rfkill: Avoid using a signed int for an unsigned variable
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=c9fcea121f9a
  - [BlueZ,5/9] shared/mainloop: Fix integer overflow
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=6cf9117bfd3f
  - [BlueZ,6/9] sdp: Fix ineffective error guard
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=bd954700e631
  - [BlueZ,7/9] obexd: Fix buffer overrun
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=1764cea5c7fd
  - [BlueZ,8/9] bap: Fix more memory leaks on error
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=cc8e6ef63509
  - [BlueZ,9/9] avdtp: Fix manipulating struct as an array
    (no matching commit)

You are awesome, thank you!