lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <751926.73238.qm@web53707.mail.re2.yahoo.com> Date: Fri, 26 Nov 2010 01:40:12 -0800 (PST) From: Nagendra Tomar <tomer_iisc@...oo.com> To: Eric Dumazet <eric.dumazet@...il.com> Cc: netdev@...r.kernel.org, davem@...emloft.net Subject: Re: [PATCH] net: Fix __inet_inherit_port() to correctly increment bsockets and num_owners --- On Fri, 26/11/10, Eric Dumazet <eric.dumazet@...il.com> wrote: > > OK so you'll have to make a proof, because current code > seems to work ;) > > ok, so I printed hashinfo->bsockets and tb->num_owners inside __inet_put_port() and I could see both of them to be -ve. All we need to do is establish and terminate a connection. I used netcat for that. The only place 'bsockets' and 'num_owners' are used is inet_csk_get_port() and the only effect that they might have is on the choice of the port to be used for binding. 'bsockets' is used as a hint to stop searching for a free port (and instead share an already used port) when we know that all the ports could be used up. 'num_owners' is used to find the port which is least shared (to balance the 'owners' list) in case we need to share a port. Since both of these are used as optimizations (in the bind path), they do not affect correctness and hence the code works even with these values not being updated correctly. Thanks, Tomar -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists