- 28 1月, 2022 40 次提交
-
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 8607954c category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Check for potential NULL pointers. Print error message if found. Thread, that leads to this commit: https://lore.kernel.org/ntfs3/227c13e3-5a22-0cba-41eb-fcaf41940711@paragon-software.com/Reported-by: NMohammad Rasim <mohammad.rasim96@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 22b05f1a category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Don't save size of attribute reparse point as size of symlink. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit cd4c76ff category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Change argument from void* to struct REPARSE_DATA_BUFFER* We copy data to buffer, so we can read it later in ntfs_read_mft. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 14a98119 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Set size for symlink, so we don't need to calculate it on the fly. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 4dbe8e44 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Rename some variables. Returned err by default is EINVAL. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 2c690788 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Now ntfs_utf16_to_nls takes length as one of arguments. If length of symlink > 255, then we tried to convert length of symlink +- some random number. Now 255 symbols limit was removed. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 9b75450d category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- In ntfs_init_fs_context we allocate memory in fc->s_fs_info. In case of failed mount we must free it in ntfs_fill_super. We can't do it in ntfs_fs_free, because ntfs_fs_free called with fc->s_fs_info == NULL. fc->s_fs_info became NULL in sget_fc. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit ce46ae0c category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Before we haven't kept prealloc for sparse files because we thought that it will speed up create / write operations. It lead to situation, when user reserved some space for sparse file, filled volume, and wasn't able to write in reserved file. With this commit we keep prealloc. Now xfstest generic/274 pass. Fixes: be71b5cb ("fs/ntfs3: Add attrib operations") Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 95dd8b2c category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We don't need ntfs_xattr_get_acl and ntfs_xattr_set_acl. There are ntfs_get_acl_ex and ntfs_set_acl_ex. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 8241fffa category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- FALLOC_FL_PUNCH_HOLE isn't allowed with normal files. Filesystem must remember info about hole, but for normal file we can only zero it and forget. Fixes: 4342306f ("fs/ntfs3: Add file operations and implementation") Now xfstests generic/016 generic/021 generic/022 pass. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit cff32466 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Make code more readable. Don't try to read zero bytes. Add warning when size of exteneded attribute exceeds limit. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit d81e06be category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We always need to lock now, because locks became smaller (see d562e901 "fs/ntfs3: Move ni_lock_dir and ni_unlock into ntfs_create_inode"). Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit b1e0c55a category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We don't need to maintain ntfs_posix_acl_release. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 35afb70d category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- This can be reason for reported panic https://lore.kernel.org/ntfs3/f9de5807-2311-7374-afb0-bc5dffb522c0@gmail.com/ Fixes: 4342306f ("fs/ntfs3: Add file operations and implementation") Reported-by: NMohammad Rasim <mohammad.rasim96@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit dbf59e2a category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Remove ntfs_sb_info members sector_size and sector_bits. Print details why mount failed. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 09f7c338 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- If we continue to work in this case, then we can corrupt fs. Fixes: 82cae269 ("fs/ntfs3: Add initialization of super block"). Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 66019837 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- This is possible because of moving lock into ntfs_create_inode. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit ba77237e category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Right now ntfs3 uses posix_acl_equiv_mode instead of posix_acl_update_mode like all other fs. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 398c35f4 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- In case of removing of xattr there must be XATTR_REPLACE flag and zero length. We already check XATTR_REPLACE in ntfs_set_ea, so now we pass XATTR_REPLACE to ntfs_set_ea. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 0bd5fdb8 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We can safely move set_cached_acl because it works with NULL acl too. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit d562e901 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Now ntfs3 locks mutex for smaller time. Theoretically in successful cases those locks aren't needed at all. But proving the same for error cases is difficult. So instead of removing them we just move them. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 6c1ee4d3 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We need to always call indx_delete_entry after indx_insert_entry if error occurred. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 82cb8753 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Some discussion has been spoken that this deprecated mount options should be removed before 5.15 lands. This driver is not never seen day light so it was decided that nls mount option has to be removed. We have always possibility to add this if needed. One possible need is example if current ntfs driver will be taken out of kernel and ntfs3 needs to support mount options what it has. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Christophe JAILLET 提交于
mainline inclusion from mainline-v5.15 commit 808bc0a8 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- There is already a 'u8 mask' defined at the top of the function. There is no need to define a new one here. Remove the useless and shadowing new 'mask' variable. Signed-off-by: NChristophe JAILLET <christophe.jaillet@wanadoo.fr> Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Christophe JAILLET 提交于
mainline inclusion from mainline-v5.15 commit d2846bf3 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- 'fnd' has been dereferenced several time before, so testing it here is pointless. Moreover, all callers of 'indx_find()' already have some error handling code that makes sure that no NULL 'fnd' is passed. So, remove the useless test. Signed-off-by: NChristophe JAILLET <christophe.jaillet@wanadoo.fr> Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 63544672 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- This allows to wait only when it's requested. It speeds up creation of hardlinks. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit 56eaeb10 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- xfstest generic/041 works with 3003 hardlinks. Because of this we raise hardlinks limit to 4000. There are no drawbacks or regressions. Theoretically we can raise all the way up to ffff, but there is no practical use for this. Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Konstantin Komarov 提交于
mainline inclusion from mainline-v5.15 commit ee9d4810 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Do not try to insert attribute if there is no room in record. Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Colin Ian King 提交于
mainline inclusion from mainline-v5.15 commit 880301bb category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Currently a failed allocation on sbi->upcase will cause an exit via the label free_sbi causing a memory leak on object opts. Fix this by re-ordering the exit paths free_opts and free_sbi so that kfree's occur in the reverse allocation order. Addresses-Coverity: ("Resource leak") Fixes: 27fac777 ("fs/ntfs3: Init spi more in init_fs_context than fill_super") Signed-off-by: NColin Ian King <colin.king@canonical.com> Reviewed-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit a0fc05a3 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Current ntfs3 rst documentation is broken. I turn table to list table as this is current Linux documentation quide line. Simple table also did not quite work in our situation as we need to span rows together. It still look quite good as text so we did not loss anything. This will also make diffing quite bit more pleasure. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 28861e3b category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Right now sb blocksize first get initiliazed in fill_super but in can be changed in helper function. It makes more sense to that this happened only in one place. Because we move this to helper function it makes more sense that s_maxbytes will also be there. I rather have every sb releted thing in fill_super, but because there is already sb releted stuff in this helper. This will have to do for now. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 0e59a87e category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Initializing should be as close as possible when we use it so that we do not need to scroll up to see what is happening. Also bdev_get_queue() can never return NULL so we do not need to check for !rq. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 0056b273 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We can survive without this tmp point upcase. So remove it we don't have so many tmp pointer in this function. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 4ea41b3e category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Drop tmp pointer bd_inode because this is only used ones in fill_super. Also we have so many initializing happening at the beginning that it is already way too much to follow. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 0cde7e81 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- We only use this in two places so we do not really need it. Also wrapper sb_rdonly() is pretty self explanatory. This will make little bit easier to read this super long variable list in the beginning of ntfs_fill_super(). Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit b4f110d6 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Use sb instead of sbi->sb in fill_super. We have sb so why not use it. Also makes code more readable. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 10b4f12c category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Remove some unnecessary variable loading. These look like copy paste work and they are not used to anything. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit bce1828f category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- In many places it is not needed to use goto out. We can just return right away. This will make code little bit more cleaner as we won't need to check error path. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 5d7d6b16 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Remove root drop when we fault out. This can never happened because when we allocate root we eather fault when no root or success. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-
由 Kari Argillander 提交于
mainline inclusion from mainline-v5.15 commit 7ea04817 category: feature bugzilla: https://gitee.com/openeuler/kernel/issues/I4G67J?from=project-issue CVE: NA ---------------------------------------------------------------------- Change EINVAL to ENOMEM when d_make_root fails because that is right errno. Signed-off-by: NKari Argillander <kari.argillander@gmail.com> Signed-off-by: NKonstantin Komarov <almaz.alexandrovich@paragon-software.com> Signed-off-by: NYin Xiujiang <yinxiujiang@kylinos.cn> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com> Acked-by: NHou Tao <houtao1@huawei.com> Acked-by: NXie XiuQi <xiexiuqi@huawei.com> Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
-