Re: [FFmpeg-devel] [PATCH] avcodec/hevc_ps: Always initialize backup in decode_vui()

2014-09-06 Thread Michael Niedermayer
On Sat, Sep 06, 2014 at 01:28:41PM +0200, Christophe Gisquet wrote: > 2014-09-06 12:40 GMT+02:00 Michael Niedermayer : > > -// Backup context in case an alternate header is detected > > -if( get_bits_left(gb) >= 66) > > -memcpy(&backup, gb, sizeof(backup)); > > +memcpy(&backup,

Re: [FFmpeg-devel] [PATCH] avcodec/hevc_ps: Always initialize backup in decode_vui()

2014-09-06 Thread Christophe Gisquet
2014-09-06 12:40 GMT+02:00 Michael Niedermayer : > -// Backup context in case an alternate header is detected > -if( get_bits_left(gb) >= 66) > -memcpy(&backup, gb, sizeof(backup)); > +memcpy(&backup, gb, sizeof(backup)); I know people consider obvious comments to waste screen

[FFmpeg-devel] [PATCH] avcodec/hevc_ps: Always initialize backup in decode_vui()

2014-09-06 Thread Michael Niedermayer
Fixes CID1237283 Signed-off-by: Michael Niedermayer --- libavcodec/hevc_ps.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c index 365652c..219af2f 100644 --- a/libavcodec/hevc_ps.c +++ b/libavcodec/hevc_ps.c @@ -526,9 +526,7