[BlueZ 00/15] Fix a number of static analysis issues #2

[Bastien Nocera <hadess@hadess.net>]

"main: Simplify variable assignment" makes a come back, moving out the
single special-case out of the function.

For "shared/gatt-client: Fix uninitialised variable usage", please verify
that this default value is correct.

Happy to iterate on any patches you feel are suboptimal. Note that the
only patches that received any sort of real-world testing are the one
mentioned above and the gdbus one.

Cheers

Bastien Nocera (15):
  main: Simplify variable assignment
  shared/ecc: Fix uninitialised variable usage
  shared/gatt-client: Fix uninitialised variable usage
  tools/mesh-cfgclient: Fix uninitialised variable usage
  test-runner: Remove unused envp
  test-runner: Fix uninitialised variable usage
  test-runner: Fix uninitialised variable usage
  shared/bap: Fix possible use-after-free
  isotest: Fix bad free
  test-runner: Fix fd leak on failure
  isotest: Fix string size expectations
  mgmt-tester: Fix non-nul-terminated string
  gdbus: Check sprintf retval
  shared/bap: Fix memory leak in error path
  android/handsfree: Check sprintf retval

 android/handsfree.c      | 14 +++++++++---
 gdbus/watch.c            | 46 +++++++++++++++++++++++++++++-----------
 src/main.c               | 19 +++++++++--------
 src/shared/bap.c         | 19 ++++++++++++-----
 src/shared/ecc.c         |  2 ++
 src/shared/gatt-client.c |  2 +-
 tools/isotest.c          | 20 +++++++++++++----
 tools/mesh-cfgclient.c   |  3 +--
 tools/mgmt-tester.c      |  8 +++++--
 tools/test-runner.c      | 15 +++++++++----
 10 files changed, 106 insertions(+), 42 deletions(-)

-- 
2.44.0