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

2017-02-17 Thread 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 : > > 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.

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

2017-02-17 Thread Carl Eugen Hoyos
2017-02-17 1:42 GMT+01:00 Michael Niedermayer : > 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.

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

2017-02-16 Thread Michael Niedermayer
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 the issue described is real but a bit hypothetical and probably has no real world eff