Hi On Mon, Jun 23, 2025 at 08:48:05PM +0800, ld y wrote: > Michael Niedermayer <mich...@niedermayer.cc> writes: > > > does the other return in config_output() not need this free too? > > > > I believe all the other return behind fir_to_phase() has called > av_free(h[longer]) > already. I would love to add this message to my next patch series.
doesnt the failure path here need the free too ? int ret = fir_to_phase(s, &h[longer], &n, &post_peak, s->phase); if (ret < 0) return ret; thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Homeopathy is like voting while filling the ballot out with transparent ink. Sometimes the outcome one wanted occurs. Rarely its worse than filling out a ballot properly.
signature.asc
Description: PGP signature
_______________________________________________ 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".