RE: [BlueZ] adapt:Set property mode failed,memory leak This patch will fix a memory leak,when set property mode,it will send a msg, but if failed,the data'memory do not free

[bluez.test.bot@gmail.com]

[-- Attachment #1: Type: text/plain, Size: 1162 bytes --]


This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
While we are preparing for reviewing the patches, we found the following
issue/warning.

Test Result:
checkbuild Failed

Outputs:
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
src/adapter.c: In function ‘property_set_mode’:
src/adapter.c:2919:2: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
 2919 |  if (mgmt_send(adapter->mgmt, opcode, adapter->dev_id, len, param,
      |  ^~
src/adapter.c:2922:3: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
 2922 |   return;
      |   ^~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:9167: src/bluetoothd-adapter.o] Error 1
make: *** [Makefile:4010: all] Error 2



---
Regards,
Linux Bluetooth