!13 xfsdump: fix memory leak

From: @liuh0329 
Reviewed-by: @swf504 
Signed-off-by: @swf504

0001-xfsdump-fix-memory-leak.patch

@@ -0,0 +1,48 @@
+From 577e51a5abdda97ffce73637d67f5f88d7e93ac7 Mon Sep 17 00:00:00 2001
+From: Pavel Reichl <preichl@redhat.com>
+Date: Thu, 14 Dec 2023 21:25:49 +0100
+Subject: xfsdump: Fix memory leak
+
+Fix memory leak found by coverity.
+
+>>>     CID 1554295:  Resource leaks  (RESOURCE_LEAK)
+>>>     Failing to save or free storage allocated by strdup(path) leaks it.
+
+Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
+Signed-off-by: Pavel Reichl <preichl@redhat.com>
+Signed-off-by: Carlos Maiolino <cem@kernel.org>
+---
+ restore/tree.c | 15 ++++++++++++++-
+ 1 file changed, 14 insertions(+), 1 deletion(-)
+
+diff --git a/restore/tree.c b/restore/tree.c
+index 6f3180f5..4707fdc9 100644
+--- a/restore/tree.c
++++ b/restore/tree.c
+@@ -4977,9 +4977,22 @@ static int
+ mkdir_r(char *path)
+ {
+ 	struct stat sbuf;
++	char *path_copy;
++	int ret;
+ 
+ 	if (stat(path, &sbuf) < 0) {
+-		if (mkdir_r(dirname(strdup(path))) < 0)
++		path_copy = strdup(path);
++		if (!path_copy) {
++			mlog(MLOG_TRACE | MLOG_ERROR | MLOG_TREE,
++				_("unable to allocate memory for a path\n"));
++			mlog_exit(EXIT_ERROR, RV_ERROR);
++			exit(1);
++		}
++
++		ret = mkdir_r(dirname(path_copy));
++		free(path_copy);
++
++		if (ret < 0)
+ 			return -1;
+ 		return mkdir(path, 0755);
+ 	}
+-- 
+cgit 1.2.3-korg
+

xfsdump.spec

@@ -1,15 +1,17 @@
 Name:      xfsdump
-Version:   3.1.9
+Version:   3.1.12
-Release:   1
+Release:   2
 Summary:   Tools create and restore for the XFS filesystem
 License:   GPLv2
 URL:       http://xfs.org
 Source0:   http://kernel.org/pub/linux/utils/fs/xfs/%{name}/%{name}-%{version}.tar.xz
 
-BuildRequires: libtool, gettext, gawk, git
+BuildRequires: libtool, gettext, gawk
 BuildRequires: xfsprogs-devel, ncurses-devel, libuuid-devel, libattr-devel
 Requires:      xfsprogs, attr
 
+Patch0:         0001-xfsdump-fix-memory-leak.patch
+
 %description
 xfsdump tools create and restore backups of directory trees on XFS filesystems.
 
@@ -21,7 +23,7 @@ Requires:       man
 This contains man files for the using of xfsdump.
 
 %prep
-%autosetup -n %{name}-%{version} -p1 -S git
+%autosetup -n %{name}-%{version} -p1
 
 %build
 %configure
@@ -48,6 +50,21 @@ mkdir -p %{buildroot}/%{_sharedstatedir}/xfsdump/inventory
 
 
 %changelog
+* Wed Mar 20 2024 liuh <liuhuan01@kylinos.cn> - 3.1.12-2
+- xfsdump: fix memory leak
+
+* Wed Feb 8 2023 wuguanghao <wuguanghao3@huawei.com> - 3.1.12-1
+- upgrade xfsdump version to 3.1.12-1
+
+* Thu Oct 20 2022 yangchenguang <yangchenguang@uniontech.com> - 3.1.11-1
+- update xfsdump version to 3.1.11-1
+
+* Mon Jul 18 2022 zhanchengbin <zhanchengbin1@huawei.com> - 3.1.10-1
+- update xfsdump version to 3.1.10-1
+
+* Fri Jul 30 2021 chenyanpanHW <chenyanpan@huawei.com> - 3.1.9-2
+- DESC: delete -S git from %autosetup, and delete BuildRequires git
+
 * Thu Jul 16 2020 wuguanghao <wuguanghao3@huawei.com> - 3.1.9-1
 - update xfsdump version to 3.1.9-1