On May 2, 2017 17:35, "Hendrik Leppkes" <[email protected]> wrote:
On Tue, May 2, 2017 at 11:25 PM, Sean McGovern <[email protected]> wrote: > On May 1, 2017 10:11, "Luca Barbato" <[email protected]> wrote: > > On 5/1/17 7:11 AM, Sean McGovern wrote: >> Bug-Id: 1047 >> CC: [email protected] >> --- >> libavcodec/aacsbr.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c >> index 99f7b08..d9bbe5e 100644 >> --- a/libavcodec/aacsbr.c >> +++ b/libavcodec/aacsbr.c >> @@ -1097,6 +1097,7 @@ int ff_decode_sbr_extension(AACContext *ac, > SpectralBandReplication *sbr, >> if (bytes_read > cnt) { >> av_log(ac->avctx, AV_LOG_ERROR, >> "Expected to read %d SBR bytes actually read %d.\n", cnt, > bytes_read); >> + sbr_turnoff(sbr); >> } >> return cnt; >> } >> > > Sounds a good solution, anybody against it? > > lu > _______________________________________________ > libav-devel mailing list > [email protected] > https://lists.libav.org/mailman/listinfo/libav-devel > > > > I'm not super happy with my patch subject though. Anyone have a better > suggestion? > This is what Alex used for the same patch some months ago: http://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=2c8a3aa 985acc906ecab37357f2798da7cb9822d - Hendrik _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel Hmmm.. Would it be OK for me to cherry-pick his patch, and just add the Bug-Id and CC ? -- Sean McG. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
