From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [RFC][PATCH 2/3] ext2: Enable atomic inode security labeling Date: Wed, 13 Jul 2005 13:50:39 -0700 Message-ID: <20050713135039.3bd7a59a.akpm@osdl.org> References: <1120829121.19035.45.camel@moss-spartans.epoch.ncsc.mil> <1120830914.19035.62.camel@moss-spartans.epoch.ncsc.mil> <1121287069.9235.25.camel@kleikamp.austin.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: sds@tycho.nsa.gov, linux-fsdevel@vger.kernel.org, viro@parcelfarce.linux.theplanet.co.uk, Ext2-devel@lists.sourceforge.net, agruen@suse.de, adilger@clusterfs.com, sct@redhat.com, jmorris@redhat.com, chrisw@osdl.org Return-path: To: Dave Kleikamp In-Reply-To: <1121287069.9235.25.camel@kleikamp.austin.ibm.com> Sender: ext2-devel-admin@lists.sourceforge.net Errors-To: ext2-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: List-Id: linux-fsdevel.vger.kernel.org Dave Kleikamp wrote: > > > + err = ext2_init_security(inode,dir); > > Won't this be unresolved if CONFIG_EXT2_FS_SECURITY is unset? > xattr_security.c won't be compiled at all. The ext3 patch has the same > problem. Yeah. I think I kicked this into shape in rc2-mm2: --- 25/fs/ext3/xattr.h~ext3-enable-atomic-inode-security-labeling-fix 2005-07-12 05:02:43.000000000 -0600 +++ 25-akpm/fs/ext3/xattr.h 2005-07-12 05:18:06.000000000 -0600 @@ -67,7 +67,6 @@ extern struct xattr_handler ext3_xattr_s extern ssize_t ext3_listxattr(struct dentry *, char *, size_t); -extern int ext3_init_security(handle_t *handle, struct inode *inode, struct inode *dir); extern int ext3_xattr_get(struct inode *, int, const char *, void *, size_t); extern int ext3_xattr_list(struct inode *, char *, size_t); extern int ext3_xattr_set(struct inode *, int, const char *, const void *, size_t, int); @@ -134,3 +133,14 @@ exit_ext3_xattr(void) #define ext3_xattr_handlers NULL # endif /* CONFIG_EXT3_FS_XATTR */ + +#ifdef CONFIG_EXT3_FS_SECURITY +extern int ext3_init_security(handle_t *handle, struct inode *inode, + struct inode *dir); +#else +static inline int ext3_init_security(handle_t *handle, struct inode *inode, + struct inode *dir) +{ + return 0; +} +#endif _ ------------------------------------------------------- This SF.Net email is sponsored by the 'Do More With Dual!' webinar happening July 14 at 8am PDT/11am EDT. We invite you to explore the latest in dual core and dual graphics technology at this free one hour event hosted by HP, AMD, and NVIDIA. To register visit http://www.hp.com/go/dualwebinar