Hi On Mon, Jun 23, 2025 at 08:48:05PM +0800, ld y wrote: > Michael Niedermayer <[email protected]> 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 [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
