shared/gatt: Fix NULL pointer dereference on destroy This patch adds pointer check to gatt_db_destroy allowing user to call it with NULL pointer. This follow convention used in shared code for destroy functions. This fix following android bluetoothd crash in a case crypto failed to initialize. ... I/ProbeModule(24651): insmod_by_dep: parse base black list error -1 E/ProbeModule(24651): insmod_by_dep: cannot find module's dependency info: [net-pf-38] I/bluetoothd(24639): bluetoothd[24640]: gatt: Failed to setup crypto ... I/DEBUG (16237): *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** I/DEBUG (16237): Build fingerprint: 'Intel/hsb/hsb:4.4.2/KOT49H/eng.android.20140514.123328:eng/test-keys' I/DEBUG (16237): Revision: '0' I/DEBUG (16237): pid: 24640, tid: 24640, name: bluetoothd-main >>> /system/bin/bluetoothd-main <<< I/DEBUG (16237): signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 00000004 ... I/DEBUG (16237): stack: I/DEBUG (16237): ff81a9dc ff81aa14 [stack] I/DEBUG (16237): ff81a9e0 00000000 I/DEBUG (16237): ff81a9e4 f75c2e06 /system/lib/libc.so (vsyslog+6) I/DEBUG (16237): ff81a9e8 f77c0cd8 /system/bin/bluetoothd-main I/DEBUG (16237): ff81a9ec f778692c /system/bin/bluetoothd-main (gatt_db_destroy+12) I/DEBUG (16237): #00 ff81a9f0 00000003 I/DEBUG (16237): ff81a9f4 f77865c0 /system/bin/bluetoothd-main (gatt_db_service_destroy) I/DEBUG (16237): ff81a9f8 f7784709 ...

Diffstat

1 files changed, 3 insertions(+), 0 deletions(-)