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

Steven Liu lq at chinaffmpeg.org
Thu Oct 10 06:40:11 EEST 2019


Signed-off-by: Steven Liu <lq at 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





More information about the ffmpeg-devel mailing list