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

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

2017-02-16 Thread Carl Eugen Hoyos
Hi! Attached patch is said to fix undefined behaviour, see ticket #6153, I cannot reproduce. Please comment, Carl Eugen From 8bbd58c465deb28737586c9292d57ab75e3de32c Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Fri, 17 Feb 2017 00:46:14 +0100 Subject: [PATCH] lavc/avpacket: Initialize