openhpi: close socket in saHpiSessionClose

Upstream issue: https://github.com/open-hpi/openhpi/issues/1918

The saHpiSessionClose doesn't close the socket and leaks the file
descriptor.

Signed-off-by: Catalin Enache <catalin.enache@windriver.com>
Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com>
Signed-off-by: Yi Zhao <yi.zhao@windriver.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
This commit is contained in:
Ovidiu Panait 2018-07-12 15:38:50 +08:00 committed by Khem Raj
parent df080427eb
commit 41c23d32ca
2 changed files with 38 additions and 0 deletions

View File

@ -0,0 +1,37 @@
From b8bc6bfdb5e7fb5b46d3a830e04632939bee6b98 Mon Sep 17 00:00:00 2001
From: Catalin Enache <catalin.enache@windriver.com>
Date: Fri, 9 Feb 2018 16:35:11 +0200
Subject: [PATCH] saHpiSessionClose: close socket
saHpiSessionClose leaks file descriptors
Upstream issue: https://github.com/open-hpi/openhpi/issues/1918
Upstream-Status: Pending
Signed-off-by: Catalin Enache <catalin.enache@windriver.com>
Signed-off-by: Yi Zhao <yi.zhao@windriver.com>
---
baselib/session.cpp | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/baselib/session.cpp b/baselib/session.cpp
index c5edfc8..4ece1b7 100644
--- a/baselib/session.cpp
+++ b/baselib/session.cpp
@@ -126,6 +126,12 @@ cSession::cSession()
cSession::~cSession()
{
+ cClientStreamSock * sock;
+ gpointer ptr = wrap_g_static_private_get( &m_sockets );
+ if ( ptr ) {
+ sock = reinterpret_cast<cClientStreamSock *>(ptr);
+ sock->Close();
+ }
wrap_g_static_private_free( &m_sockets );
}
--
2.10.2

View File

@ -44,6 +44,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.gz \
file://fix-narrowing-warning.patch \
file://0001-plugins-Check-for-PTHREAD_RECURSIVE_MUTEX_INITIALIZE.patch \
file://0001-ipmidirect-Replace-__STRING.patch \
file://0001-session-close-socket.patch \
"
SRC_URI[md5sum] = "4718b16e0f749b5ad214a9b04f45dd23"