[FFmpeg-devel] [PATCH]lavc/avpacket: Initialize a variable in error path

[Michael Niedermayer]

On Fri, Feb 17, 2017 at 10:53:26AM +0100, Carl Eugen Hoyos wrote:
> 2017-02-17 1:42 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:
> > On Fri, Feb 17, 2017 at 12:49:05AM +0100, Carl Eugen Hoyos wrote:
> >> Hi!
> >>
> >> Attached patch is said to fix undefined behaviour, see ticket #6153,
> >> I cannot reproduce.
> >>
> >> Please comment, Carl Eugen
> >
> > patch is ok
> 
> Patch pushed and backported.
> 

> > the issue described is real but a bit hypothetical and probably has
> > no real world effect
> 
> I don't understand how the issue can be triggered...

a compiler can optimize it out, i would expect it to need -O0
also iam not sure what ubsan exactly considers an issue, depending on
that it may be random if it can be reproduced
didnt try or see the exact ubsan error


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170217/b99972bc/attachment.sig>