netdev - Re: [GIT]: Networking

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]

Date:	Tue, 16 Jun 2009 02:56:59 -0700 (PDT)
From:	David Miller <davem@...emloft.net>
To:	mingo@...e.hu
Cc:	alan@...rguk.ukuu.org.uk, torvalds@...ux-foundation.org,
	akpm@...ux-foundation.org, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [GIT]: Networking

From: Ingo Molnar <mingo@...e.hu>
Date: Tue, 16 Jun 2009 11:48:13 +0200

> I think David's larger point was that what looked to me like more 
> widespread breakage was in fact limited to [a]x25, a comparatively 
> rarely used (and rarely tested) corner of the kernel.
> 
> I turned them off and that solved the problems here. (and i suspect 
> the real fix wont be far away either) Thanks everyone.

This code path has been broken, basically, forever.  It does a
lock_sock() from a timer, which has that might_sleep() check, and
essentially always has.

It would have warned this way years ago if someone had triggered it.

The code has been this way forever, so something else changed to start
triggering this on you.

I still am curious what in the world opens those sockets for you,
nothing normal should be doing so.

Can you satisfy my curiosity by turning X25 back on and booting
with this patch?  Thanks.

diff --git a/net/x25/af_x25.c b/net/x25/af_x25.c
index ed80af8..5214a38 100644
--- a/net/x25/af_x25.c
+++ b/net/x25/af_x25.c
@@ -494,6 +494,10 @@ static int x25_create(struct net *net, struct socket *sock, int protocol)
 	struct x25_sock *x25;
 	int rc = -ESOCKTNOSUPPORT;
 
+#if 1
+	printk(KERN_INFO "[%s:%d]: Creates X25 socket.\n",
+	       current->comm, current->pid);
+#endif
 	if (net != &init_net)
 		return -EAFNOSUPPORT;
 
--
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