[FFmpeg-devel] [PATCH] avformat/oggparseflac: check init_get_bits' result

2023-05-30 Thread Paul Arzelier
iting this patch after I reported the bug! Signed-off-by: Paul Arzelier --- libavformat/oggparseflac.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/oggparseflac.c b/libavformat/oggparseflac.c index eef6e09927..557440d94b 100644 --- a/libavformat/oggparsef

Re: [FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-28 Thread Paul Arzelier
Hopefuly, everything is okay right now :) Le 27/01/2017 à 17:42, wm4 a écrit : On Fri, 27 Jan 2017 17:32:08 +0100 Paul Arzelier wrote: From 227d7d1f4b5665f824900cf2b14863621180dd1c Mon Sep 17 00:00:00 2001 From: Polochon-street Date: Fri, 27 Jan 2017 16:49:41 +0100 Subject: [PATCH] Ignore

Re: [FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-27 Thread Paul Arzelier
+0100 Michael Niedermayer wrote: On Thu, Jan 26, 2017 at 02:29:21PM +0100, Paul Arzelier wrote: Alright, attached is the last version (I hope!) Paul Le 26/01/2017 à 13:43, wm4 a écrit : On Thu, 26 Jan 2017 13:32:08 +0100 Paul Arzelier wrote: From d84648e1990ad3a12462dfb76990dc7036f5f08

Re: [FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-26 Thread Paul Arzelier
Alright, attached is the last version (I hope!) Paul Le 26/01/2017 à 13:43, wm4 a écrit : On Thu, 26 Jan 2017 13:32:08 +0100 Paul Arzelier wrote: From d84648e1990ad3a12462dfb76990dc7036f5f082 Mon Sep 17 00:00:00 2001 From: Polochon-street Date: Thu, 26 Jan 2017 13:25:22 +0100 Subject

Re: [FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-26 Thread Paul Arzelier
which seems to be a bit more strict when it comes down to tags). Le 26/01/2017 à 13:10, wm4 a écrit : On Thu, 26 Jan 2017 12:55:15 +0100 Paul Arzelier wrote: From a3dc6068fb06722aacea56365f948afdb8df841f Mon Sep 17 00:00:00 2001 From: Paul Arzelier Date: Thu, 26 Jan 2017 12:51:33 +0100 Su

Re: [FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-26 Thread Paul Arzelier
You're right, I made a patch for libavformat/utils.c instead. I modified a bit Ben's version and kept only the "spec-compliant flac files do not support ID3 tags" warning, I hope it's okay with you. Paul Le 26/01/2017 à 11:24, wm4 a écrit : On Thu, 26 Jan 2017 11:

Re: [FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-26 Thread Paul Arzelier
Oops, I don't know what went wrong, but now it seems to work. I've attached the working patched, sorry! Le 26/01/2017 à 02:12, Michael Niedermayer a écrit : On Wed, Jan 25, 2017 at 08:33:53PM +0100, Paul Arzelier wrote: Hi all, Would it be possible to continue the discussion b

[FFmpeg-devel] [PATCH] Ignore duplicate ID3 tags if vorbis tags exist

2017-01-25 Thread Paul Arzelier
uot;, or does it only raise more questions? Regards, Paul >From b10a32fecf05b17f6e824b662b7db07bff195d2d Mon Sep 17 00:00:00 2001 From: Paul Arzelier Date: Wed, 25 Jan 2017 18:55:52 +0100 Subject: [PATCH] Fixed behavior when id3 tags were found on FLAC files Originally-by: Ben Boeckel --- libavformat/flacdec.c | 31 +++