Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

2015-10-07 Thread wm4
On Tue, 6 Oct 2015 20:13:58 -0300 James Almer wrote: > On 10/6/2015 5:33 PM, Paul B Mahol wrote: > > you set aligned number of samples before calling get_buffer and after > > that changes > > frame->nb_samples to actual number of samples. > > > > Alternatively IIRC default 16

[FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

2015-10-06 Thread James Almer
Since AVFrame.extended_data is apparently not padded, simd functions could in some cases overread, so make the decoder use a temp buffer unconditionally. Signed-off-by: James Almer --- libavcodec/alac.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-)

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

2015-10-06 Thread Paul B Mahol
On 10/6/15, James Almer wrote: > Since AVFrame.extended_data is apparently not padded, simd functions > could in some cases overread, so make the decoder use a temp buffer > unconditionally. > > Signed-off-by: James Almer > --- > libavcodec/alac.c | 18

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

2015-10-06 Thread James Almer
On 10/6/2015 4:40 PM, Paul B Mahol wrote: > On 10/6/15, James Almer wrote: >> Since AVFrame.extended_data is apparently not padded, simd functions >> could in some cases overread, so make the decoder use a temp buffer >> unconditionally. >> >> Signed-off-by: James Almer

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

2015-10-06 Thread Paul B Mahol
On 10/6/15, James Almer wrote: > On 10/6/2015 4:40 PM, Paul B Mahol wrote: >> On 10/6/15, James Almer wrote: >>> Since AVFrame.extended_data is apparently not padded, simd functions >>> could in some cases overread, so make the decoder use a temp buffer >>>