Re: /dev/urandom uses uninit bytes, leaks user data

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: John Reiser <jreiser@...>
Cc: Matt Mackall <mpm@...>, <linux-kernel@...>, <security@...>
Date: Friday, December 14, 2007 - 7:23 pm

On Fri, Dec 14, 2007 at 12:45:23PM -0800, John Reiser wrote:

Um, no.  Just seeding the pool with your own data won't help, since
that still won't tell you the initial contents of the pool.  And if
you know the initial contents of the pool, then you've broken root.
And being able to steal from the pool also assumes that you've broken
into the system; it is never, ever exported to userspace, even if
you're root (unless you use something like /dev/kmem).  Furthermore,
if you don't know the previous contents of the pool, you'll never be
able to recover the information, either now or five years in the
future, since information is XOR'ed into the pool.


How about wrapping it in a #ifdef CONFIG_UML, which is the only way
you can use Valgrind?  The memset will slow down things unnecessarily,
and mixing in the unknown previous contents of the stack (a) doesn't
hurt, and (b) could make life more complicated for an attacker.

            	      	    	      - Ted
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: /dev/urandom uses uninit bytes, leaks user data, John Reiser, (Fri Dec 14, 4:45 pm)
Re: /dev/urandom uses uninit bytes, leaks user data, Theodore Tso, (Fri Dec 14, 7:23 pm)
Re: /dev/urandom uses uninit bytes, leaks user data, John Reiser, (Fri Dec 14, 8:30 pm)
Re: /dev/urandom uses uninit bytes, leaks user data, Herbert Xu, (Sat Dec 15, 3:13 am)
Re: /dev/urandom uses uninit bytes, leaks user data, Matt Mackall, (Sat Dec 15, 12:30 pm)