netdev - Re: [Lguest] [PATCH 4/5] lguest: use KVM hypercalls

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 for Android: free password hash cracker in your pocket

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

Message-ID: <m11vruovu5.fsf@fess.ebiederm.org>
Date:	Wed, 15 Apr 2009 07:06:10 -0700
From:	ebiederm@...ssion.com (Eric W. Biederman)
To:	Herbert Xu <herbert@...dor.apana.org.au>
Cc:	Patrick McHardy <kaber@...sh.net>,
	Matias Zabaljauregui <zabaljauregui@...il.com>, odie@...aau.dk,
	Rusty Russell <rusty@...tcorp.com.au>, lguest@...abs.org,
	virtualization@...ts.osdl.org,
	"David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org,
	Christian Borntraeger <borntraeger@...ibm.com>
Subject: Re: [Lguest] [PATCH 4/5] lguest: use KVM hypercalls

Herbert Xu <herbert@...dor.apana.org.au> writes:

> On Wed, Apr 15, 2009 at 06:35:58AM -0700, Eric W. Biederman wrote:
>> 
>> Because as far as I can tell we would just leak that refcount.
>> 
>> The poll code does not appear to call back into any of the file
>> methods when it frees itself from the wait queue.
>
> OK my suggestion was stupid.
>
> But I still don't see how this race is possible at all.
>
> So process A has a tun fd open and is spinning in poll(2).  Now
> process B comes along and deletes that tun device.  Process A's
> fd should have a netdev reference that keeps the device and
> associated structures alive.
>
> Oh I see what's going on.  We're automatically detaching the
> device in uninit.  This is just wrong.  Just because process B
> deleted the netdev, process A should not be involutarily detached.
>
> Does anything actually rely on this behaviour?

Yes.

There is the boring rmmod case that has always existed.

There is more interesting case of moving your tap device
into another network namespace.

In which case there is the possibility of the network namespace
exiting and destroying all of the virtual network devices before
we close the file handle.

I implemented the rtnetlink methods so we can test the uninit
behavior without going through all sorts of hoops.

Eric
--
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