[FFmpeg-devel] [PATCH] libavformat/sdp: VPS is not a mandatory HEVC codec parameter.

[koushd at gmail.com]

From: Koushik Dutta <koushd at gmail.com>

Many IP cameras do not place VPS in the SDP, and only send it
in the stream. The RTSP SDP only contains SPS/PPS.

Signed-off-by: Koushik Dutta <koushd at gmail.com>
---
 libavformat/sdp.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/libavformat/sdp.c b/libavformat/sdp.c
index 28490d77f3..64fe9d61ac 100644
--- a/libavformat/sdp.c
+++ b/libavformat/sdp.c
@@ -301,7 +301,9 @@ static int extradata2psets_hevc(AVFormatContext *fmt, const AVCodecParameters *p
             pos += len;
         }
     }
-    if (!ps_pos[0] || !ps_pos[1] || !ps_pos[2])
+
+    // VPS is not mandatory
+    if (!ps_pos[1] || !ps_pos[2])
         goto err;
 
     psets = av_mallocz(MAX_PSET_SIZE);
@@ -314,6 +316,8 @@ static int extradata2psets_hevc(AVFormatContext *fmt, const AVCodecParameters *p
 
     for (i = 0; i < 3; i++) {
         pos = ps_pos[i];
+        if (!pos)
+            continue;
 
         if (i > 0)
             av_strlcat(psets, "; ", MAX_PSET_SIZE);
-- 
2.39.5 (Apple Git-154)