linux-kernel - Re: [PATCH v5] futex: Fix regression with read only mappings

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: <alpine.LFD.2.02.1107252125270.2848@ionos>
Date:	Mon, 25 Jul 2011 21:28:55 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Shawn Bohrer <sbohrer@...advisors.com>
cc:	Darren Hart <dvhart@...ux.intel.com>,
	KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
	peterz@...radead.org, eric.dumazet@...il.com,
	david@...advisors.com, linux-kernel@...r.kernel.org,
	zvonler@...advisors.com, hughd@...gle.com, mingo@...e.hu
Subject: Re: [PATCH v5] futex: Fix regression with read only mappings

On Mon, 25 Jul 2011, Shawn Bohrer wrote:
> On Thu, Jun 30, 2011 at 11:21:32AM -0500, Shawn Bohrer wrote:
> > commit 7485d0d3758e8e6491a5c9468114e74dc050785d (futexes: Remove rw
> > parameter from get_futex_key()) in 2.6.33 fixed two problems:  First, It
> >  
> > -	ret = get_futex_key(uaddr2, flags & FLAGS_SHARED, &key2);
> > +	ret = get_futex_key(uaddr2, flags & FLAGS_SHARED, &key2, VERIFY_WRITE);
> >  	if (unlikely(ret != 0))
> >  		goto out;
> >  
> 
> Thomas,
> 
> What do I need to do to get this patch accepted?

First of all, please use proper quoting. It's a nuisance to page down
10 times to find the interesting new text of a mail :)

Seriously, I'm sorry, it fell from my radar. Thanks for the
reminder. It's on top of my tomorrow morning (brain usable) todo list
now.

Thanks,

	tglx
--
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/