On Sun, Nov 01, 2015 at 05:01:37PM -0500, Xu Cang wrote: > diff --git a/fs/ext4/file.c b/fs/ext4/file.c > index 113837e..1c0086a 100644 > --- a/fs/ext4/file.c > +++ b/fs/ext4/file.c > @@ -295,7 +295,7 @@ static int ext4_file_mmap(struct file *file, > struct vm_area_struct *vma) > if (ext4_encrypted_inode(inode)) { > int err = ext4_get_encryption_info(inode); > if (err) > - return 0; > + return -EACCES; Shouldn't this return err here? --D > if (ext4_encryption_info(inode) == NULL) > return -ENOKEY; > } > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html