Ok. I'll move this and the third patch to one commit/patch and also change
the condition when rtsp_src can't be allocated to also free dest_addr. Good
catch.
On Wed, Feb 26, 2025, 3:53 AM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
> Rashad Tatum:
> > ---
> > libavformat/rtsp.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
> > index 0c65f8d1a4..ac17717195 100644
> > --- a/libavformat/rtsp.c
> > +++ b/libavformat/rtsp.c
> > @@ -478,6 +478,7 @@ static void sdp_parse_line(AVFormatContext *s,
> SDPParseState *s1,
> > sdp_ip_str);
> >
> > }
> > + av_freep(&sdp_ip_str);
> > break;
> > case 's':
> > av_dict_set(&s->metadata, "title", p, 0);
> > @@ -702,6 +703,7 @@ static void sdp_parse_line(AVFormatContext *s,
> SDPParseState *s1,
> > }
> > }
> > }
> > + av_freep(&dest_addr);
> > } else {
> > if (rt->server_type == RTSP_SERVER_WMS)
> > ff_wms_parse_sdp_a_line(s, p);
>
> Your first patch introduces a memleak; the stuff you allocated there
> should be freed in the patch, not fixed up in a later patch.
> (Also note that there must not be a leak in any error path, which is not
> true when your patchset is applied: dest_addr leaks when rtsp_src can't
> be allocated.)
>
> - 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".
