* [FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error
@ 2025-04-01 23:36 Kacper Michajłow
2025-04-02 21:24 ` Michael Niedermayer
0 siblings, 1 reply; 3+ messages in thread
From: Kacper Michajłow @ 2025-04-01 23:36 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Kacper Michajłow
Signed-off-by: Kacper Michajłow <kasper93@gmail.com>
---
tools/target_swr_fuzzer.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tools/target_swr_fuzzer.c b/tools/target_swr_fuzzer.c
index f154a11632..59fa24af64 100644
--- a/tools/target_swr_fuzzer.c
+++ b/tools/target_swr_fuzzer.c
@@ -79,7 +79,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
char out_layout_string[256];
uint8_t * ain[SWR_CH_MAX];
uint8_t *aout[SWR_CH_MAX];
- uint8_t *out_data;
+ uint8_t *out_data = NULL;
int in_sample_nb;
int out_sample_nb = size;
int count;
@@ -145,9 +145,8 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
count = swr_convert(swr, aout, out_sample_nb, (const uint8_t **)ain, in_sample_nb);
- av_freep(&out_data);
-
end:
+ av_freep(&out_data);
swr_free(&swr);
return 0;
--
2.45.1
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error
2025-04-01 23:36 [FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error Kacper Michajłow
@ 2025-04-02 21:24 ` Michael Niedermayer
0 siblings, 0 replies; 3+ messages in thread
From: Michael Niedermayer @ 2025-04-02 21:24 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1.1: Type: text/plain, Size: 409 bytes --]
On Wed, Apr 02, 2025 at 01:36:30AM +0200, Kacper Michajłow wrote:
> Signed-off-by: Kacper Michajłow <kasper93@gmail.com>
> ---
> tools/target_swr_fuzzer.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 251 bytes --]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 3+ messages in thread
* [FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error
@ 2025-04-01 11:20 Kacper Michajłow
0 siblings, 0 replies; 3+ messages in thread
From: Kacper Michajłow @ 2025-04-01 11:20 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Kacper Michajłow
---
tools/target_swr_fuzzer.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tools/target_swr_fuzzer.c b/tools/target_swr_fuzzer.c
index f154a11632..59fa24af64 100644
--- a/tools/target_swr_fuzzer.c
+++ b/tools/target_swr_fuzzer.c
@@ -79,7 +79,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
char out_layout_string[256];
uint8_t * ain[SWR_CH_MAX];
uint8_t *aout[SWR_CH_MAX];
- uint8_t *out_data;
+ uint8_t *out_data = NULL;
int in_sample_nb;
int out_sample_nb = size;
int count;
@@ -145,9 +145,8 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
count = swr_convert(swr, aout, out_sample_nb, (const uint8_t **)ain, in_sample_nb);
- av_freep(&out_data);
-
end:
+ av_freep(&out_data);
swr_free(&swr);
return 0;
--
2.45.1
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-04-02 21:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-04-01 23:36 [FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error Kacper Michajłow
2025-04-02 21:24 ` Michael Niedermayer
-- strict thread matches above, loose matches on Subject: below --
2025-04-01 11:20 Kacper Michajłow
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git