From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> queue_remove should not check the data pointer since both queue_push_head and queue_push_tail don't check it is possible to have NULL pointers in the queue therefore queue_remove shall be able to remove them. --- src/shared/queue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/shared/queue.c b/src/shared/queue.c index 3bdc1ec..3ca3ca6 100644 --- a/src/shared/queue.c +++ b/src/shared/queue.c @@ -238,7 +238,7 @@ bool queue_remove(struct queue *queue, void *data) { struct queue_entry *entry, *prev; - if (!queue || !data) + if (!queue) return false; for (entry = queue->head, prev = NULL; entry; -- 1.9.3 -- 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