On Tue, 9 Nov 2010 13:15:07 -0600 [email protected] wrote: > --- a/fs/cifs/inode.c > +++ b/fs/cifs/inode.c > @@ -688,10 +688,9 @@ int cifs_get_inode_info(struct inode **pinode, > > #ifdef CONFIG_CIFS_EXPERIMENTAL > /* fill in 0777 bits from ACL */ > - if (cifs_sb->mnt_cifs_flags & CIFS_MOUNT_CIFS_ACL) { > - cFYI(1, "Getting mode bits from ACL"); > - cifs_acl_to_fattr(cifs_sb, &fattr, *pinode, full_path, pfid); > - } > + if (cifs_sb->mnt_cifs_flags & CIFS_MOUNT_CIFS_ACL) > + rc = cifs_acl_to_fattr(cifs_sb, &fattr, *pinode, full_path, > + pfid);
Do you want to return an error from cifs_get_inode_info when fetching the ACL fails? I think you either need to goto cgii_exit if rc != 0, or keep ignoring the error code like you do today. If the function is going to return an error, then there's no real point in doing the later stuff (the iget, etc). -- Jeff Layton <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
