netdev - Re: SCTP control socket question

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1206625405.2109.7.camel@iris.sw.ru>
Date:	Thu, 27 Mar 2008 16:43:25 +0300
From:	"Denis V. Lunev" <den@...nvz.org>
To:	Vlad Yasevich <vladislav.yasevich@...com>
Cc:	Netdev List <netdev@...r.kernel.org>,
	lksctp-developers@...ts.sourceforge.net
Subject: Re: SCTP control socket question

On Thu, 2008-03-27 at 09:15 -0400, Vlad Yasevich wrote:
> Denis V. Lunev wrote:
> > On Thu, 2008-03-27 at 08:48 -0400, Vlad Yasevich wrote:
> >> Denis V. Lunev wrote:
> >>> Hello, Vlad!
> >>>
> >>> I have read SCTP sources and found that SCTP control socket
> >>> (sctp_ctl_socket) remains hashed unlike similar staff in UDP/TCP etc.
> >>>
> >>> Am I wrong, that it should not be hashed or not?
> >> Why do you believe that the socket is hashed?  That socket is only
> >> allocated and referenced.  It's should not be in any hash tables
> >> since we hold a global reference on it.
> > 
> > sctp_ctl_sock_init
> >   __sock_create
> >     inet(6)_create
> >         sk->sk_prot->hash(sk);
> > 
> > For this purpose sk->sk_prot->unhash is explicitly called for ICMP/TCP
> > etc cases.
> > 
> > Regards,
> > 	Den
> > 
> 
> Ahhh...  ;-)
> 
> struct proto sctp_prot = {
> ...
> 	.hash        =  sctp_hash,
> ...
> }
> 
> static void sctp_hash(struct sock *sk)
> {
>         /* STUB */
> }
> 
> 
> SCTP currently does't do any hashing on the sockets.  It hashes SCTP structures
> that live under the socket.

will you mind against this?

View attachment "1.txt" of type "text/x-patch" (656 bytes)