diff options
| author | Michael Niedermayer <michaelni@gmx.at> | 2015-06-03 22:39:27 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2015-08-20 14:29:11 +0200 |
| commit | 8498cf60b209f3038efb8cd88bc6fdfd74bd1666 (patch) | |
| tree | dd2aaaa853eda3530334b746685036df8835eb3b | |
| parent | 39430866566a087919a31641fa24ecfdd4d7719a (diff) | |
| download | ffmpeg-8498cf60b209f3038efb8cd88bc6fdfd74bd1666.tar.gz | |
Revert "avformat/rtpenc: check av_packet_get_side_data() return, fix null ptr dereference"
This was simply wrong
Found-by: Martin Storsjö
This reverts commit 5d8e4f6da03c0342157e6ac7fab1a8ac3a87a8b0.
(cherry picked from commit 3e34b7498f14c04baadde1700a6f73a7e9e86fa6)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
| -rw-r--r-- | libavformat/rtpenc.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/libavformat/rtpenc.c b/libavformat/rtpenc.c index 9c587d2dc5..8c5ce9763e 100644 --- a/libavformat/rtpenc.c +++ b/libavformat/rtpenc.c @@ -566,10 +566,6 @@ static int rtp_write_packet(AVFormatContext *s1, AVPacket *pkt) const uint8_t *mb_info = av_packet_get_side_data(pkt, AV_PKT_DATA_H263_MB_INFO, &mb_info_size); - if (!mb_info) { - av_log(s1, AV_LOG_ERROR, "failed to allocate side data\n"); - return AVERROR(ENOMEM); - } ff_rtp_send_h263_rfc2190(s1, pkt->data, size, mb_info, mb_info_size); break; } |