Re: [PATCH v4] ax25: Fix refcount imbalance on inbound connections
From: Dan Carpenter <hidden>
Date: 2024-05-29 14:34:26
Also in:
linux-hams
[ Sorry for the delay on sending this. My email daemon messed up so all my outgoing mail has been going to /dev/null for the past few days. Resending everything... -dan ] On Tue, May 28, 2024 at 12:06:39PM -0400, Lars Kellogg-Stedman wrote:
On Tue, May 28, 2024 at 11:40:38AM GMT, Paolo Abeni wrote:quoted
Note that the fixes tag above is still wrong - the hash must be 12 chars long, see:I had spotted that, thanks. Just waiting to see if there are any real change requests before re-submitting.
Okay. It looks good to me. Sorry for all the headache. 1) The Fixes tag points to the wrong commit, though, right? The one you have here doesn't make sense and it doesn't match the bisect. 2) Can we edit the commitmessage a bit to say include what you wrote about "but rather bind/accept" being paired. We increment in bind and we should increment in accept as well. It's the same. 3) The other thing that I notice is that Duoming dropped part of his commit when he resent v6. https://lore.kernel.org/all/5c61fea1b20f3c1596e4fb46282c3dedc54513a3.1715065005.git.duoming@zju.edu.cn/ (local) That part of the commit was correct. Maybe it wasn't necessary but it feels right and it's more readable and it's obviously harmless. I can resend that. regards, dan carpenter