On Wed, 26 Aug 2020 19:18:09 +0800 wangyunjian <wangyunjian at huawei.com> wrote: > if (tx_qconf) > free(tx_qconf); > - if (rx_qconf) > - free(rx_qconf); free accepts a NULL pointer so current practice is to avoid useless if() tests before free.