From mboxrd@z Thu Jan 1 00:00:00 1970 From: Trond Myklebust Subject: Re: [patch 11/14] vfs: move executable checking into ->permission() Date: Wed, 21 May 2008 15:26:24 -0400 Message-ID: <1211397984.7486.38.camel@localhost> References: <20080521171458.077908538@szeredi.hu> <20080521171557.399850407@szeredi.hu> <1211393793.7486.15.camel@localhost> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, hch@infradead.org, viro@ZenIV.linux.org.uk, linux-kernel@vger.kernel.org To: Miklos Szeredi Return-path: Received: from pat.uio.no ([129.240.10.15]:55249 "EHLO pat.uio.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751625AbYEUT02 (ORCPT ); Wed, 21 May 2008 15:26:28 -0400 In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, 2008-05-21 at 21:09 +0200, Miklos Szeredi wrote: > No, we don't want to deny read or write (that's up to the filesystem > how it handles it), just want to deny execute if no x bits are set in > the mode. OK, but when I see something with the name 'exec_permission()', I assume that it is going to check for whether or not I have execute permission. If that is not the case, then can we please either change the function, or change the name? Cheers Trond