[FFmpeg-devel] [PATCH v1 8/8] avformat/sapdec: check av_strdup() return value and fix memleak

[Steven Liu]

Signed-off-by: Steven Liu <l...@chinaffmpeg.org>
---
 libavformat/sapdec.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/libavformat/sapdec.c b/libavformat/sapdec.c
index eddeddeaea..4cce700b40 100644
--- a/libavformat/sapdec.c
+++ b/libavformat/sapdec.c
@@ -142,6 +142,10 @@ static int sap_read_header(AVFormatContext *s)
         }
 
         sap->sdp = av_strdup(&recvbuf[pos]);
+        if (!sap->sdp) {
+            ret = AVERROR(ENOMEM);
+            goto fail;
+        }
         break;
     }
 
@@ -183,6 +187,7 @@ static int sap_read_header(AVFormatContext *s)
     return 0;
 
 fail:
+    av_freep(&sap->sdp);
     sap_read_close(s);
     return ret;
 }
-- 
2.15.1



_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".