On 09/04/14 00:07, Vittorio Giovara wrote: > On Sat, Mar 1, 2014 at 11:35 PM, Vittorio Giovara > <[email protected]> wrote: >> On Sun, Feb 9, 2014 at 4:59 AM, Luca Barbato <[email protected]> wrote: >>> On 09/02/14 02:45, Vittorio Giovara wrote: >>>> On Sun, Jan 12, 2014 at 7:55 PM, Diego Biurrun <[email protected]> wrote: >>>>> From: Sean McGovern <[email protected]> >>>>> >>>>> Signed-off-by: Diego Biurrun <[email protected]> >>>>> --- >>>>> >>>>> I fixed the bits that I noted during my review. >>>>> >>>>> libavformat/oggdec.c | 71 >>>>> +++++++++++++++++++++++++++++++++++++++++--------- >>>>> 1 file changed, 59 insertions(+), 12 deletions(-) >>>>> >>>> >>>> Apparently ok, any more comments? >>> >>> I'm quite sure there is at least one place that requires some other kind >>> of undo in the fail path, remind me to look at it again. >> >> Ping. > > PIng. >
Shouldn't break stuff more than it would do in the current state, probably a run of valgrind injecting memory faults (the wiki documents how thanks to Derek) might be nice. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
