> dont.b...@gmail.com:
> > From: walle <dont.b...@gmail.com>
> >
> > lock screen or other permission notification in win7 or
> > higher windows version would case BitBlt return false,
> > and gdigrab_read_packet will return AVERROR(EIO) and do nothing to
> > allocated packet data, then memory leak happend. It's necessary to
> > release packet data before return from gdigrab_read_packet.
> > ---
> > libavdevice/gdigrab.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/libavdevice/gdigrab.c b/libavdevice/gdigrab.c
> > index f4444406fa..b69494c98d 100644
> > --- a/libavdevice/gdigrab.c
> > +++ b/libavdevice/gdigrab.c
> > @@ -576,6 +576,7 @@ static int gdigrab_read_packet(AVFormatContext *s1,
> > AVPacket *pkt)
> > clip_rect.bottom - clip_rect.top,
> > source_hdc,
> > clip_rect.left, clip_rect.top, SRCCOPY | CAPTUREBLT)) {
> > + av_packet_unref(pkt);
> > WIN32_API_ERROR("Failed to capture image");
> > return AVERROR(EIO);
> > }
> >
> What version did you use when encountering this leak? This leak should
> have been fixed in bae8844e35147f92e612a9e0b44e939a293e5bc9.
Your reply is really useful, thanks, I'm using the old 3.4.7 version. And would
you please tell me how to reply the email with "> " ahead of each line, I'm
using an email client and have to add them manually.
dont.b...@gmail.com
From: Andreas Rheinhardt
Date: 2020-10-22 21:52
To: ffmpeg-devel
Subject: Re: [FFmpeg-devel] [PATCH] avdevice/gdigrab: Memory leak if BitBlt
return false.
dont.b...@gmail.com:
> From: walle <dont.b...@gmail.com>
>
> lock screen or other permission notification in win7 or
> higher windows version would case BitBlt return false,
> and gdigrab_read_packet will return AVERROR(EIO) and do nothing to
> allocated packet data, then memory leak happend. It's necessary to
> release packet data before return from gdigrab_read_packet.
> ---
> libavdevice/gdigrab.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/libavdevice/gdigrab.c b/libavdevice/gdigrab.c
> index f4444406fa..b69494c98d 100644
> --- a/libavdevice/gdigrab.c
> +++ b/libavdevice/gdigrab.c
> @@ -576,6 +576,7 @@ static int gdigrab_read_packet(AVFormatContext *s1,
> AVPacket *pkt)
> clip_rect.bottom - clip_rect.top,
> source_hdc,
> clip_rect.left, clip_rect.top, SRCCOPY | CAPTUREBLT)) {
> + av_packet_unref(pkt);
> WIN32_API_ERROR("Failed to capture image");
> return AVERROR(EIO);
> }
>
What version did you use when encountering this leak? This leak should
have been fixed in bae8844e35147f92e612a9e0b44e939a293e5bc9.
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
