Re: [FFmpeg-devel] [PATCH] ffplay: Separate AVFrame from AVSubtitle

2014-09-26 Thread Michael Niedermayer
On Sat, Sep 27, 2014 at 12:38:28AM +0200, Marton Balint wrote: > > On Fri, 26 Sep 2014, Michael Niedermayer wrote: > > >This is more robust and should avoid false positives in coverity > >That is if the type is ever mixed up, after this patch that would > >just be a null pointer dereference inste

Re: [FFmpeg-devel] [PATCH] ffplay: Separate AVFrame from AVSubtitle

2014-09-26 Thread Marton Balint
On Fri, 26 Sep 2014, Michael Niedermayer wrote: This is more robust and should avoid false positives in coverity That is if the type is ever mixed up, after this patch that would just be a null pointer dereference instead of a random possibly out of array access Signed-off-by: Michael Niederma

[FFmpeg-devel] [PATCH] ffplay: Separate AVFrame from AVSubtitle

2014-09-26 Thread Michael Niedermayer
This is more robust and should avoid false positives in coverity That is if the type is ever mixed up, after this patch that would just be a null pointer dereference instead of a random possibly out of array access Signed-off-by: Michael Niedermayer --- ffplay.c | 11 +-- 1 file change