[Ocfs2-devel] [PATCH] Ocfs2: do not allow fallocate on dir file
Sunil Mushran
sunil.mushran at oracle.com
Wed Mar 9 18:27:52 PST 2011
On 03/09/2011 03:01 PM, Mark Fasheh wrote:
> On Fri, Mar 04, 2011 at 03:10:33PM +0800, Li Dongyang wrote:
>> @@ -1870,6 +1870,11 @@ static int __ocfs2_change_file_space(struct file *file, struct inode *inode,
>> goto out_inode_unlock;
>> }
>>
>> + if (!S_ISREG(inode->i_mode)) {
>> + ret = -EINVAL;
>> + goto out_inode_unlock;
>> + }
> You might want to move the check into ocfs2_fallocate to mirror what
> ocfs2_change_file_space() does. Otherwise, looks like a good catch.
>
Tristan pointed out that this check was already in ocfs2_change_file_space().
So this patch should not be required. Am I missing something?
More information about the Ocfs2-devel
mailing list