import libXpm-3.5.12-8.el8
This commit is contained in:
parent
69a8cf83c7
commit
7f83cb7f3f
@ -0,0 +1,40 @@
|
|||||||
|
From e00066fb973a1796dd3989e356e17c8b51add521 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Alan Coopersmith <alan.coopersmith@oracle.com>
|
||||||
|
Date: Sun, 30 Sep 2018 15:09:29 -0700
|
||||||
|
Subject: [PATCH 1/3] After fdopen(), use fclose() instead of close() in error
|
||||||
|
path
|
||||||
|
|
||||||
|
Found by Oracle's Parfait 2.2 static analyzer:
|
||||||
|
|
||||||
|
Error: File Leak
|
||||||
|
File Leak [file-ptr-leak]:
|
||||||
|
Leaked File fp
|
||||||
|
at line 94 of lib/libXpm/src/RdFToBuf.c in function 'XpmReadFileToBuffer
|
||||||
|
'.
|
||||||
|
fp initialized at line 86 with fdopen
|
||||||
|
fp leaks when len < 0 at line 92.
|
||||||
|
|
||||||
|
Introduced-by: commit 8b3024e6871ce50b34bf2dff924774bd654703bc
|
||||||
|
|
||||||
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
||||||
|
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
|
||||||
|
---
|
||||||
|
src/RdFToBuf.c | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/src/RdFToBuf.c b/src/RdFToBuf.c
|
||||||
|
index 69e3347d24f2..1b386f81fde3 100644
|
||||||
|
--- a/src/RdFToBuf.c
|
||||||
|
+++ b/src/RdFToBuf.c
|
||||||
|
@@ -90,7 +90,7 @@ XpmReadFileToBuffer(
|
||||||
|
}
|
||||||
|
len = stats.st_size;
|
||||||
|
if (len < 0 || len >= SIZE_MAX) {
|
||||||
|
- close(fd);
|
||||||
|
+ fclose(fp);
|
||||||
|
return XpmOpenFailed;
|
||||||
|
}
|
||||||
|
ptr = (char *) XpmMalloc(len + 1);
|
||||||
|
--
|
||||||
|
2.23.0
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
Summary: X.Org X11 libXpm runtime library
|
Summary: X.Org X11 libXpm runtime library
|
||||||
Name: libXpm
|
Name: libXpm
|
||||||
Version: 3.5.12
|
Version: 3.5.12
|
||||||
Release: 7%{?dist}
|
Release: 8%{?dist}
|
||||||
License: MIT
|
License: MIT
|
||||||
Group: System Environment/Libraries
|
Group: System Environment/Libraries
|
||||||
URL: http://www.x.org
|
URL: http://www.x.org
|
||||||
@ -13,6 +13,8 @@ BuildRequires: autoconf automake libtool
|
|||||||
BuildRequires: gettext
|
BuildRequires: gettext
|
||||||
BuildRequires: pkgconfig(xext) pkgconfig(xt) pkgconfig(xau)
|
BuildRequires: pkgconfig(xext) pkgconfig(xt) pkgconfig(xau)
|
||||||
|
|
||||||
|
Patch0: 0001-After-fdopen-use-fclose-instead-of-close-in-error-pa.patch
|
||||||
|
|
||||||
%description
|
%description
|
||||||
X.Org X11 libXpm runtime library
|
X.Org X11 libXpm runtime library
|
||||||
|
|
||||||
@ -26,6 +28,7 @@ X.Org X11 libXpm development package
|
|||||||
|
|
||||||
%prep
|
%prep
|
||||||
%setup -q
|
%setup -q
|
||||||
|
%patch0 -p1 -b .covscan
|
||||||
|
|
||||||
%build
|
%build
|
||||||
autoreconf -v --install --force
|
autoreconf -v --install --force
|
||||||
@ -59,6 +62,9 @@ rm -f $RPM_BUILD_ROOT%{_libdir}/*.la
|
|||||||
#%{_mandir}/man1/*.1x*
|
#%{_mandir}/man1/*.1x*
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Mon Dec 09 2019 Benjamin Tissoires <benjamin.tissoires@redhat.com> 3.5.12-8%{?dist}
|
||||||
|
- add covscan fixes (#1602606)
|
||||||
|
|
||||||
* Thu Jul 05 2018 Adam Jackson <ajax@redhat.com> - 3.5.12-7
|
* Thu Jul 05 2018 Adam Jackson <ajax@redhat.com> - 3.5.12-7
|
||||||
- Drop useless %%defattr
|
- Drop useless %%defattr
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user