"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
