Hi Marcin, On Wednesday 24 of September 2014 13:37:46 Marcin Kraglak wrote: > It may cause NULL pointer dereference. > --- > android/tester-gatt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/android/tester-gatt.c b/android/tester-gatt.c > index 42f3733..f30d0eb 100644 > --- a/android/tester-gatt.c > +++ b/android/tester-gatt.c > @@ -545,7 +545,7 @@ static void gatt_cid_hook_cb(const void *data, uint16_t len, void *user_data) > } > queue_pop_head(t_data->pdus); > gatt_pdu = queue_pop_head(t_data->pdus); > - if (!gatt_pdu->data) > + if (!gatt_pdu) Shouldn't this be: if (!gatt_pdu || !gatt_pdu->data) ? > break; > > bthost_send_cid(bthost, cid_data->handle, cid_data->cid, > -- Best regards, Szymon Janc -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html