On Thu, Nov 04, 2021 at 12:38:23PM +0800, Wu Bo wrote:> if (!reply_head) {> - nlmsg_free(reply);> goto out;> }Please also drop the now pointless braces.> out:> + if (reply)> + nlmsg_free(reply);Please just use a different label for just unlocking vs also freeingthe reply.