Re: [FFmpeg-devel] [PATCH] avcodec/dxvenc: Use proper alignment, write endian-independent output

2024-02-27 Thread Andreas Rheinhardt
Sean McGovern: > Hi, > > On Mon, Feb 26, 2024 at 6:48 PM Andreas Rheinhardt > wrote: >> >> Fixes the dxv3enc-dxt1 FATE test with UBSan. >> >> Signed-off-by: Andreas Rheinhardt >> --- >> I have not actually tested whether the output is actually wrong >> on BE systems. Would be nice if someone

Re: [FFmpeg-devel] [PATCH] avcodec/dxvenc: Use proper alignment, write endian-independent output

2024-02-27 Thread Sean McGovern
Hi, On Mon, Feb 26, 2024 at 6:48 PM Andreas Rheinhardt wrote: > > Fixes the dxv3enc-dxt1 FATE test with UBSan. > > Signed-off-by: Andreas Rheinhardt > --- > I have not actually tested whether the output is actually wrong > on BE systems. Would be nice if someone could. > > libavcodec/dxvenc.c

Re: [FFmpeg-devel] [PATCH] avcodec/dxvenc: Use proper alignment, write endian-independent output

2024-02-27 Thread James Almer
On 2/26/2024 8:50 PM, Andreas Rheinhardt wrote: Fixes the dxv3enc-dxt1 FATE test with UBSan. Signed-off-by: Andreas Rheinhardt --- I have not actually tested whether the output is actually wrong on BE systems. Would be nice if someone could. libavcodec/dxvenc.c | 4 ++-- 1 file changed, 2

[FFmpeg-devel] [PATCH] avcodec/dxvenc: Use proper alignment, write endian-independent output

2024-02-26 Thread Andreas Rheinhardt
Fixes the dxv3enc-dxt1 FATE test with UBSan. Signed-off-by: Andreas Rheinhardt --- I have not actually tested whether the output is actually wrong on BE systems. Would be nice if someone could. libavcodec/dxvenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git