linux-kernel - Re: BUG: Bad page map in process udevd (anon

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: <20110220.115355.59672016.davem@davemloft.net>
Date:	Sun, 20 Feb 2011 11:53:55 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	torvalds@...ux-foundation.org
Cc:	ebiederm@...ssion.com, mhocko@...e.cz, mingo@...e.hu,
	linux-mm@...ck.org, linux-kernel@...r.kernel.org,
	eric.dumazet@...il.com
Subject: Re: BUG: Bad page map in process udevd (anon_vma: (null)) in
 2.6.38-rc4

From: Linus Torvalds <torvalds@...ux-foundation.org>
Date: Sat, 19 Feb 2011 22:15:23 -0800

> So unregister_netdevice_many() should always return with the list
> empty and destroyed. There is no valid use of a list of netdevices
> after you've unregistered them.
> 
> Now, dev_deactivate_many() actually has uses of that list after
> they've been de-activated (__dev_close_many will deactivate them, and
> then after that do the whole ndo_stop dance too, so I guess all (two)
> callers of that function need to get rid of their list manually. So I
> think your patch to sch_generic.c is good, but I really think the
> semantics of unregister_netdevice_many() should just be changed.

The key, as Eric D. mentioned, is the moment we changed the "scope" of
this list.

Previously all manipulations and use were guareded entirely inside of
holding the RTNL mutex.

The commit that introduced this regression allowed the list to be
"live" across RTNL mutex drop/re-grab.

Anyways, Eric B.'s patch (which I've just added to net-2.6) should
handle the known remaining cases, and as Eric D. said we should do one
more audit to make sure we got them all now.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/