Re: [PATCH net v2 2/2] ax25: fix potential reference counting leak in ax25_addr_ax25dev

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

To: Markus Elfring <Markus.Elfring@xxxxxx>
Subject: Re: [PATCH net v2 2/2] ax25: fix potential reference counting leak in ax25_addr_ax25dev
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Fri, 3 May 2024 14:38:00 +0300
Cc: Duoming Zhou <duoming@xxxxxxxxxx>, linux-hams@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>, Eric Dumazet <edumazet@xxxxxxxxxx>, Jakub Kicinski <kuba@xxxxxxxxxx>, Jörg Reuter <jreuter@xxxxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>, LKML <linux-kernel@xxxxxxxxxxxxxxx>, Lars Kellogg-Stedman <lars@xxxxxxxxxx>, Simon Horman <horms@xxxxxxxxxx>
In-reply-to: <6eac7fc4-9ade-41bb-a861-d7f339b388f6@web.de>
References: <cover.1714690906.git.duoming@zju.edu.cn> <74e840d98f2bfc79c6059993b2fc1ed3888faba4.1714690906.git.duoming@zju.edu.cn> <6eac7fc4-9ade-41bb-a861-d7f339b388f6@web.de>

Yeah, it's true that we should delete the curly braces around the if
block.  Otherwise checkpatch.pl -f will complain.

The commit message is fine as-is.  Please stop nit-picking.

regards,
dan carpenter

Follow-Ups:
- Re: [v2 2/2] ax25: fix potential reference counting leak in ax25_addr_ax25dev
  - From: Markus Elfring

References:
- Re: [PATCH net v2 2/2] ax25: fix potential reference counting leak in ax25_addr_ax25dev
  - From: Markus Elfring

Prev by Date: Re: [PATCH net v2 1/2] ax25: change kfree in ax25_dev_free to ax25_dev_free
Next by Date: Re: [PATCH][next] btrfs: Fix spelling mistake "decompresssed" -> "decompressed"
Previous by thread: Re: [PATCH net v2 2/2] ax25: fix potential reference counting leak in ax25_addr_ax25dev
Next by thread: Re: [v2 2/2] ax25: fix potential reference counting leak in ax25_addr_ax25dev
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]