On 5/26/2014 3:56 PM, Nidhi Makhijani wrote:
> @@ -427,13 +427,17 @@ rdt_parse_sdp_line (AVFormatContext *s, int st_index,
> if ((err = av_reallocp(&rdt->rmst,
> count * sizeof(*rdt->rmst))) < 0)
> {
> rdt->nb_rmst = 0;
> - return err;
> + return AVERROR(ENOMEM);
> }
Should be a separate patch.
> rdt->rmst[s->streams[n]->index] = ff_rm_alloc_rmstream();
> + if (!rdt->rmst[s->streams[n]->index]) {
> + av_freep(&rdt->rmst);
> + return AVERROR(ENOMEM);
> + }
Won't this leak any existing members of rmst?
- Derek
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
