[FFmpeg-cvslog] rtpdec: Remove unnecessary checks

Martin Storsjö git at videolan.org
Tue Feb 24 23:44:34 CET 2015


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Tue Feb 24 13:30:12 2015 +0200| [12251f997bbc0abb93be39c51021e6d404ca385f] | committer: Martin Storsjö

rtpdec: Remove unnecessary checks

The free function of a depacketizer won't be called if data is NULL.

Signed-off-by: Martin Storsjö <martin at martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=12251f997bbc0abb93be39c51021e6d404ca385f
---

 libavformat/rtpdec_h263_rfc2190.c |    2 --
 libavformat/rtpdec_latm.c         |    2 --
 2 files changed, 4 deletions(-)

diff --git a/libavformat/rtpdec_h263_rfc2190.c b/libavformat/rtpdec_h263_rfc2190.c
index a7e8287..ae6e38e 100644
--- a/libavformat/rtpdec_h263_rfc2190.c
+++ b/libavformat/rtpdec_h263_rfc2190.c
@@ -41,8 +41,6 @@ struct PayloadContext {
 
 static void h263_free_context(PayloadContext *data)
 {
-    if (!data)
-        return;
     if (data->buf) {
         uint8_t *p;
         avio_close_dyn_buf(data->buf, &p);
diff --git a/libavformat/rtpdec_latm.c b/libavformat/rtpdec_latm.c
index 9758efa..2da0fbf 100644
--- a/libavformat/rtpdec_latm.c
+++ b/libavformat/rtpdec_latm.c
@@ -33,8 +33,6 @@ struct PayloadContext {
 
 static void latm_free_context(PayloadContext *data)
 {
-    if (!data)
-        return;
     if (data->dyn_buf) {
         uint8_t *p;
         avio_close_dyn_buf(data->dyn_buf, &p);



More information about the ffmpeg-cvslog mailing list