Re: [PATCH RESEND] nbd: code clean for nbd_genl_status()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2021/11/4 16:48, Christoph Hellwig wrote:
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.

ok. Will do in v2.
  out:
+	if (reply)
+		nlmsg_free(reply);

Please just use a different label for just unlocking vs also freeing
the reply.
.
ok. Will do in v2.

Thanks.

Wu Bo




[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux