Thread (16 messages) 16 messages, 5 authors, 8d ago

Re: [PATCH net v2 2/4] net: netlink: don't set nsid on local notifications

From: Nicolas Dichtel <hidden>
Date: 2026-05-21 14:25:40
Also in: linux-kselftest, lkml

Le 21/05/2026 à 16:00, Jiri Benc a écrit :
On Thu, 21 May 2026 14:36:12 +0200, Nicolas Dichtel wrote:
quoted
I still don't think that this is the right "fix". The app is broken. Even after
this patch, the bug could be easily triggered again by a third party.
There is nothing wrong with assigning a self-nsid. It would be a lot more robust
for the app to assign itself a self-nsid when it starts.
On the other hand, does the patch break anything in practice (as
opposed to in theory)? It makes live of several apps simpler, which is
not a bad goal.
I'm not against the patch, it just look like a workaround.
I'm trying to understand how NETLINK_LISTEN_ALL_NSID is used (in fact, why it is
used if the app doesn't "understand" NSIDs).
The only scenario where this would introduce incompatible behavior is
an app that self-assigns a self-nsid and expects to see it. That looks
Yes, I thought about this.
quite stretched, doesn't it?
It does.

Regards,
Nicolas.
Not that I have a strong opinion about this, though.

 Jiri
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help