MINOR: queue: Replace if() + abort() with BUG_ON()
see 5cd4bbd7a ("BUG/MAJOR: threads/queue: Fix thread-safety issues on the queues management")
diff --git a/src/queue.c b/src/queue.c
index b1be766..002b94b 100644
--- a/src/queue.c
+++ b/src/queue.c
@@ -567,14 +567,13 @@
struct pendconn *p;
int is_unlinked;
- if (unlikely(!strm->pend_pos)) {
- /* unexpected case because it is called by the stream itself and
- * only the stream can release a pendconn. So it is only
- * possible if a pendconn is released by someone else or if the
- * stream is supposed to be queued but without its associated
- * pendconn. In both cases it is a bug! */
- abort();
- }
+ /* unexpected case because it is called by the stream itself and
+ * only the stream can release a pendconn. So it is only
+ * possible if a pendconn is released by someone else or if the
+ * stream is supposed to be queued but without its associated
+ * pendconn. In both cases it is a bug! */
+ BUG_ON(!strm->pend_pos);
+
p = strm->pend_pos;
/* note below : we need to grab the queue's lock to check for emptiness