Michael Niedermayer <michael at 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. Thanks, Lidong