* [FFmpeg-devel] [PATCH] examples: Add av_freep to avoid potential memory leak
@ 2025-08-02 23:28 Jiasheng Jiang
2025-08-03 23:31 ` Michael Niedermayer
0 siblings, 1 reply; 2+ messages in thread
From: Jiasheng Jiang @ 2025-08-02 23:28 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Jiasheng Jiang
Add av_freep() to free avio_ctx_buffer if avio_alloc_context fails
to avoid potential memory leak.
Fixes: 5fc4dea39c ("examples: add avio_reading.c example")
Signed-off-by: Jiasheng Jiang <jiashengjiangcool@gmail.com>
---
doc/examples/avio_read_callback.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/doc/examples/avio_read_callback.c b/doc/examples/avio_read_callback.c
index dffc061d9f..0a299b4c73 100644
--- a/doc/examples/avio_read_callback.c
+++ b/doc/examples/avio_read_callback.c
@@ -96,6 +96,7 @@ int main(int argc, char *argv[])
avio_ctx = avio_alloc_context(avio_ctx_buffer, avio_ctx_buffer_size,
0, &bd, &read_packet, NULL, NULL);
if (!avio_ctx) {
+ av_freep(&avio_ctx_buffer);
ret = AVERROR(ENOMEM);
goto end;
}
--
2.25.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] 2+ messages in thread
* Re: [FFmpeg-devel] [PATCH] examples: Add av_freep to avoid potential memory leak
2025-08-02 23:28 [FFmpeg-devel] [PATCH] examples: Add av_freep to avoid potential memory leak Jiasheng Jiang
@ 2025-08-03 23:31 ` Michael Niedermayer
0 siblings, 0 replies; 2+ messages in thread
From: Michael Niedermayer @ 2025-08-03 23:31 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1.1: Type: text/plain, Size: 573 bytes --]
On Sat, Aug 02, 2025 at 11:28:48PM +0000, Jiasheng Jiang wrote:
> Add av_freep() to free avio_ctx_buffer if avio_alloc_context fails
> to avoid potential memory leak.
>
> Fixes: 5fc4dea39c ("examples: add avio_reading.c example")
> Signed-off-by: Jiasheng Jiang <jiashengjiangcool@gmail.com>
> ---
> doc/examples/avio_read_callback.c | 1 +
> 1 file changed, 1 insertion(+)
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
What is money laundering? Its paying someone and not telling the government.
[-- 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] 2+ messages in thread
end of thread, other threads:[~2025-08-03 23:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-02 23:28 [FFmpeg-devel] [PATCH] examples: Add av_freep to avoid potential memory leak Jiasheng Jiang
2025-08-03 23:31 ` Michael Niedermayer
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