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 freeing the reply.