From 6eb01f8ffc9ba7915ffff67a6b8e4ced2ce7e0fe Mon Sep 17 00:00:00 2001
From: ejurgensen <espenjurgensen@gmail.com>
Date: Tue, 3 Nov 2020 23:44:22 +0100
Subject: [PATCH] [outputs] Remove incorrect NULL return if packet buffer
 seqnum is 0

Since seqnum wraps around it should be ok to call rtp_packet_get() even when
session->seqnum is 0.
---
 src/outputs/rtp_common.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/outputs/rtp_common.c b/src/outputs/rtp_common.c
index e749393e..bdbb00a7 100644
--- a/src/outputs/rtp_common.c
+++ b/src/outputs/rtp_common.c
@@ -204,8 +204,11 @@ rtp_packet_get(struct rtp_session *session, uint16_t seqnum)
   uint16_t delta;
   size_t idx;
 
-  if (!session->seqnum || !session->pktbuf_len)
-    return NULL;
+  if (session->pktbuf_len == 0)
+    {
+      DPRINTF(E_DBG, L_PLAYER, "Seqnum %" PRIu16 " requested, but buffer is empty\n", seqnum);
+      return NULL;
+    }
 
   last = session->seqnum - 1;
   first = session->seqnum - session->pktbuf_len;