This commit is contained in:
Kaleb S. KEITHLEY 2017-10-23 09:51:44 -04:00
parent a0a907a7d6
commit 202c34e682
2 changed files with 45 additions and 1 deletions

View File

@ -0,0 +1,39 @@
From 87bd25b64ae34cce95e87e724acfeab4c13d60a4 Mon Sep 17 00:00:00 2001
From: Sanju Rakonde <srakonde@redhat.com>
Date: Wed, 18 Oct 2017 08:06:24 +0530
Subject: [PATCH] glusterfsd: Dereferencing the null pointer
Problem: When control reaches to out, one of (iobref, iobuf, frame) can
be null.for iobref, iobuf iobref_unref() and iobuf_unref() functions
are called respectively, which are using GF_VALIDATE_OR_GOTO(), so
there won't be null pointer dereference. But for frame without null
checking STACK_DESTROY(frame->root) is called causing null pointer
dereference.
Fix: adding a line for null checking, the function
STACK_DESTROY(frame->root) is called only when frame is not null.
Change-Id: I3a6684c11fb7b694b81d6ad4fec3bced5562ad88
BUG: 1503394
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
---
glusterfsd/src/gf_attach.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/glusterfsd/src/gf_attach.c b/glusterfsd/src/gf_attach.c
index 05119e4f4..3f248292d 100644
--- a/glusterfsd/src/gf_attach.c
+++ b/glusterfsd/src/gf_attach.c
@@ -144,7 +144,8 @@ out:
iobref_unref (iobref);
iobuf_unref (iobuf);
- STACK_DESTROY (frame->root);
+ if (frame)
+ STACK_DESTROY (frame->root);
if (rpc_status != 0) {
fprintf (stderr, "got error %d on RPC\n", rpc_status);
--
2.13.5

View File

@ -176,7 +176,7 @@ Summary: Distributed File System
%if ( 0%{_for_fedora_koji_builds} ) %if ( 0%{_for_fedora_koji_builds} )
Name: glusterfs Name: glusterfs
Version: 3.12.2 Version: 3.12.2
Release: 1%{?prereltag:.%{prereltag}}%{?dist} Release: 2%{?prereltag:.%{prereltag}}%{?dist}
%else %else
Name: @PACKAGE_NAME@ Name: @PACKAGE_NAME@
Version: @PACKAGE_VERSION@ Version: @PACKAGE_VERSION@
@ -195,6 +195,7 @@ Source8: glusterfsd.init
%else %else
Source0: @PACKAGE_NAME@-@PACKAGE_VERSION@.tar.gz Source0: @PACKAGE_NAME@-@PACKAGE_VERSION@.tar.gz
%endif %endif
Patch0: 0001-glusterfsd-Dereferencing-the-null-pointer.patch
BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX) BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)
@ -661,6 +662,7 @@ This package provides the glusterfs server daemon.
%prep %prep
%setup -q -n %{name}-%{version}%{?prereltag} %setup -q -n %{name}-%{version}%{?prereltag}
%patch0 -p1
%build %build
%if ( 0%{?rhel} && 0%{?rhel} < 6 ) %if ( 0%{?rhel} && 0%{?rhel} < 6 )
@ -1383,6 +1385,9 @@ exit 0
%endif %endif
%changelog %changelog
* Mon Oct 23 2017 Kaleb S. KEITHLEY <kkeithle[at]redhat.com> - 3.12.2-2
- 3.12.2, bz #1504256
* Fri Oct 13 2017 Kaleb S. KEITHLEY <kkeithle[at]redhat.com> - 3.12.2-1 * Fri Oct 13 2017 Kaleb S. KEITHLEY <kkeithle[at]redhat.com> - 3.12.2-1
- 3.12.2 GA - 3.12.2 GA