diff mbox

[FFmpeg-devel] Revert "avutil/frame: Disallow zero sized frame side data"

Message ID 20170301120211.29147-1-michael@niedermayer.cc
State Accepted
Commit 55196e5d10af7c295352bb9a2d7565efe07ce46b
Headers show

Commit Message

Michael Niedermayer March 1, 2017, 12:02 p.m. UTC 
Found a case where we use size==0, the other related commits
remain needed, and should be sufficient to fix the original issue

This reverts commit 7e4f32f4e4b93c95dcc872cb844c5548e69f352e.
---
 libavutil/frame.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Michael Niedermayer March 2, 2017, 10:40 p.m. UTC | #1 
On Wed, Mar 01, 2017 at 01:02:11PM +0100, Michael Niedermayer wrote:
> Found a case where we use size==0, the other related commits
> remain needed, and should be sufficient to fix the original issue
> 
> This reverts commit 7e4f32f4e4b93c95dcc872cb844c5548e69f352e.
> ---
>  libavutil/frame.c | 2 --
>  1 file changed, 2 deletions(-)

applied

[...]
diff mbox

Patch

diff --git a/libavutil/frame.c b/libavutil/frame.c
index 69ee240758..8811dcdcfe 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -668,8 +668,6 @@  AVFrameSideData *av_frame_new_side_data(AVFrame *frame,
                                         enum AVFrameSideDataType type,
                                         int size)
 {
-    if (size <= 0)
-        return NULL;
 
     return frame_new_side_data(frame, type, av_buffer_alloc(size));
 }