<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:宋体;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:DengXian;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"\@宋体";
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"\@等线";
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"Segoe UI";
panose-1:2 11 5 2 4 2 4 2 2 3;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:宋体;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
{mso-style-name:msonormal;
mso-margin-top-alt:auto;
margin-right:0cm;
mso-margin-bottom-alt:auto;
margin-left:0cm;
font-size:12.0pt;
font-family:宋体;}
p.gmail-m3980449368408844904gmail-m-1109814905353783812gmail-m3903278274190134145msolistparagraph, li.gmail-m3980449368408844904gmail-m-1109814905353783812gmail-m3903278274190134145msolistparagraph, div.gmail-m3980449368408844904gmail-m-1109814905353783812gmail-m3903278274190134145msolistparagraph
{mso-style-name:gmail-m_3980449368408844904gmail-m-1109814905353783812gmail-m3903278274190134145msolistparagraph;
mso-margin-top-alt:auto;
margin-right:0cm;
mso-margin-bottom-alt:auto;
margin-left:0cm;
font-size:12.0pt;
font-family:宋体;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:DengXian;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:DengXian;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 90.0pt 72.0pt 90.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="ZH-CN" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Hi,
<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">How about this patch? I see there is a failed test, is that related to my change?<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">cynthia<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"><o:p> </o:p></span></p>
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Raghavendra Gowdappa <rgowdapp@redhat.com>
<br>
<b>Sent:</b> Thursday, May 09, 2019 12:13 PM<br>
<b>To:</b> Zhou, Cynthia (NSB - CN/Hangzhou) <cynthia.zhou@nokia-sbell.com><br>
<b>Cc:</b> Amar Tumballi Suryanarayan <atumball@redhat.com>; gluster-devel@gluster.org<br>
<b>Subject:</b> Re: [Gluster-devel] glusterfsd memory leak issue found after enable ssl<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
<div>
<p class="MsoNormal"><span lang="EN-US">Thanks!!<o:p></o:p></span></p>
</div>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US">On Thu, May 9, 2019 at 8:34 AM Zhou, Cynthia (NSB - CN/Hangzhou) <<a href="mailto:cynthia.zhou@nokia-sbell.com">cynthia.zhou@nokia-sbell.com</a>> wrote:<o:p></o:p></span></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Hi,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Ok, It is posted to
</span><span lang="EN-US"><a href="https://review.gluster.org/#/c/glusterfs/+/22687/" target="_blank">https://review.gluster.org/#/c/glusterfs/+/22687/</a><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Raghavendra
Gowdappa <<a href="mailto:rgowdapp@redhat.com" target="_blank">rgowdapp@redhat.com</a>>
<br>
<b>Sent:</b> Wednesday, May 08, 2019 7:35 PM<br>
<b>To:</b> Zhou, Cynthia (NSB - CN/Hangzhou) <<a href="mailto:cynthia.zhou@nokia-sbell.com" target="_blank">cynthia.zhou@nokia-sbell.com</a>><br>
<b>Cc:</b> Amar Tumballi Suryanarayan <<a href="mailto:atumball@redhat.com" target="_blank">atumball@redhat.com</a>>;
<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a><br>
<b>Subject:</b> Re: [Gluster-devel] glusterfsd memory leak issue found after enable ssl</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">On Wed, May 8, 2019 at 1:29 PM Zhou, Cynthia (NSB - CN/Hangzhou) <<a href="mailto:cynthia.zhou@nokia-sbell.com" target="_blank">cynthia.zhou@nokia-sbell.com</a>>
wrote:<o:p></o:p></span></p>
</div>
<blockquote style="border:none;border-left:solid windowtext 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0cm;margin-bottom:5.0pt;border-color:currentcolor currentcolor currentcolor rgb(204,204,204)">
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Hi 'Milind Changire' ,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">The leak is getting more and more clear to me now. the unsolved memory leak is because of in gluterfs version 3.12.15
(in my env)the ssl context is a shared one, while we do ssl_acept, ssl will allocate some read/write buffer to ssl object, however, ssl_free in socket_reset or fini function of socket.c, the buffer is returened back to ssl context free list instead of completely
freed.</span><span lang="EN-US"><o:p></o:p></span></p>
</div>
</div>
</blockquote>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">Thanks Cynthia for your efforts in identifying and fixing the leak. If you post a patch to gerrit, I'll be happy to merge it and get the fix into the codebase.<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<blockquote style="border:none;border-left:solid windowtext 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0cm;margin-bottom:5.0pt;border-color:currentcolor currentcolor currentcolor rgb(204,204,204)">
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">So following patch is able to fix the memory leak issue completely.(created for gluster master branch)</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">--- a/rpc/rpc-transport/socket/src/socket.c<br>
+++ b/rpc/rpc-transport/socket/src/socket.c<br>
@@ -446,6 +446,7 @@ ssl_setup_connection_postfix(rpc_transport_t *this)<br>
gf_log(this->name, GF_LOG_DEBUG,<br>
"SSL verification succeeded (client: %s) (server: %s)",<br>
this->peerinfo.identifier, this->myinfo.identifier);<br>
+ X509_free(peer);<br>
return gf_strdup(peer_CN);<br>
<br>
/* Error paths. */<br>
@@ -1157,7 +1158,21 @@ __socket_reset(rpc_transport_t *this)<br>
memset(&priv->incoming, 0, sizeof(priv->incoming));<br>
<br>
event_unregister_close(this->ctx->event_pool, priv->sock, priv->idx);<br>
-<br>
+ if(priv->use_ssl&& priv->ssl_ssl)<br>
+ {<br>
+ gf_log(this->name, GF_LOG_TRACE,<br>
+ "clear and reset for socket(%d), free ssl ",<br>
+ priv->sock);<br>
+ if(priv->ssl_ctx)<br>
+ {<br>
+ SSL_CTX_free(priv->ssl_ctx);<br>
+ priv->ssl_ctx = NULL;<br>
+ }<br>
+ SSL_shutdown(priv->ssl_ssl);<br>
+ SSL_clear(priv->ssl_ssl);<br>
+ SSL_free(priv->ssl_ssl);<br>
+ priv->ssl_ssl = NULL;<br>
+ }<br>
priv->sock = -1;<br>
priv->idx = -1;<br>
priv->connected = -1;<br>
@@ -4675,6 +4690,21 @@ fini(rpc_transport_t *this)<br>
pthread_mutex_destroy(&priv->out_lock);<br>
pthread_mutex_destroy(&priv->cond_lock);<br>
pthread_cond_destroy(&priv->cond);<br>
+ if(priv->use_ssl&& priv->ssl_ssl)<br>
+ {<br>
+ gf_log(this->name, GF_LOG_TRACE,<br>
+ "clear and reset for socket(%d), free ssl ",<br>
+ priv->sock);<br>
+ if(priv->ssl_ctx)<br>
+ {<br>
+ SSL_CTX_free(priv->ssl_ctx);<br>
+ priv->ssl_ctx = NULL;<br>
+ }<br>
+ SSL_shutdown(priv->ssl_ssl);<br>
+ SSL_clear(priv->ssl_ssl);<br>
+ SSL_free(priv->ssl_ssl);<o:p></o:p></span></p>
<div>
<div style="border:none;border-top:solid windowtext 1.0pt;padding:3.0pt 0cm 0cm 0cm;border-color:currentcolor">
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Zhou,
Cynthia (NSB - CN/Hangzhou) <br>
<b>Sent:</b> Monday, May 06, 2019 2:12 PM<br>
<b>To:</b> 'Amar Tumballi Suryanarayan' <<a href="mailto:atumball@redhat.com" target="_blank">atumball@redhat.com</a>><br>
<b>Cc:</b> 'Milind Changire' <<a href="mailto:mchangir@redhat.com" target="_blank">mchangir@redhat.com</a>>; '<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a>' <<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a>><br>
<b>Subject:</b> RE: [Gluster-devel] glusterfsd memory leak issue found after enable ssl</span><span lang="EN-US"><o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Hi,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">From our test valgrind and libleak all blame ssl3_accept</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">///////////////////////////from valgrind attached to glusterfds///////////////////////////////////////////</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">==16673== 198,720 bytes in 12 blocks are definitely lost in loss record 1,114 of 1,123<br>
==16673== at 0x4C2EB7B: malloc (vg_replace_malloc.c:299)<br>
==16673== by 0x63E1977: CRYPTO_malloc (in /usr/lib64/<u><span style="color:blue">libcrypto.so.1.0.2p</span></u>)<br>
==16673== by 0xA855E0C: ssl3_setup_write_buffer (in /usr/lib64/<u><span style="color:blue">libssl.so.1.0.2p</span></u>)<br>
==16673== by 0xA855E77: ssl3_setup_buffers (in /usr/lib64/<u><span style="color:blue">libssl.so.1.0.2p</span></u>)<br>
==16673== by 0xA8485D9: <span style="color:red">ssl3_accept</span> (in /usr/lib64/<u><span style="color:blue">libssl.so.1.0.2p</span></u>)<br>
==16673== by 0xA610DDF: ssl_complete_connection (socket.c:400)<br>
==16673== by 0xA617F38: ssl_handle_server_connection_attempt (socket.c:2409)<br>
==16673== by 0xA618420: socket_complete_connection (socket.c:2554)<br>
==16673== by 0xA618788: socket_event_handler (socket.c:2613)<br>
==16673== by 0x4ED6983: event_dispatch_epoll_handler (event-epoll.c:587)<br>
==16673== by 0x4ED6C5A: event_dispatch_epoll_worker (event-epoll.c:663)<br>
==16673== by 0x615C5D9: start_thread (in /usr/lib64/<u><span style="color:blue"><a href="http://libpthread-2.27.so" target="_blank">libpthread-2.27.so</a></span></u>)<br>
==16673==<br>
==16673== 200,544 bytes in 12 blocks are definitely lost in loss record 1,115 of 1,123<br>
==16673== at 0x4C2EB7B: malloc (vg_replace_malloc.c:299)<br>
==16673== by 0x63E1977: CRYPTO_malloc (in /usr/lib64/<u><span style="color:blue">libcrypto.so.1.0.2p</span></u>)<br>
==16673== by 0xA855D12: ssl3_setup_read_buffer (in /usr/lib64/<u><span style="color:blue">libssl.so.1.0.2p</span></u>)<br>
==16673== by 0xA855E68: ssl3_setup_buffers (in /usr/lib64/<u><span style="color:blue">libssl.so.1.0.2p</span></u>)<br>
==16673== by 0xA8485D9: <span style="color:red">ssl3_accept</span> (in /usr/lib64/<u><span style="color:blue">libssl.so.1.0.2p</span></u>)<br>
==16673== by 0xA610DDF: ssl_complete_connection (socket.c:400)<br>
==16673== by 0xA617F38: ssl_handle_server_connection_attempt (socket.c:2409)<br>
==16673== by 0xA618420: socket_complete_connection (socket.c:2554)<br>
==16673== by 0xA618788: socket_event_handler (socket.c:2613)<br>
==16673== by 0x4ED6983: event_dispatch_epoll_handler (event-epoll.c:587)<br>
==16673== by 0x4ED6C5A: event_dispatch_epoll_worker (event-epoll.c:663)<br>
==16673== by 0x615C5D9: start_thread (in /usr/lib64/<u><span style="color:blue"><a href="http://libpthread-2.27.so" target="_blank">libpthread-2.27.so</a></span></u>)<br>
==16673==<br>
valgrind --leak-check=f<o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">////////////////////////////////////with libleak attached to glusterfsd/////////////////////////////////////////</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">callstack[2419] expires. count=1 size=224/224 alloc=362 free=350<br>
/home/robot/libleak/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libleak.so(malloc+0x25</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f1460604065]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(CRYPTO_malloc+0x58</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ecd9978]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(EVP_DigestInit_ex+0x2a9</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed95749]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libssl.so.10(ssl3_digest_cached_records+0x11d</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145abb6ced]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libssl.so.10(</span></u><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:red">ssl3_accept</span></u><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">+0xc8f</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145abadc4f]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(ssl_complete_connection+0x5e</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae00f3a]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(+0xc16d</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae0816d]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(+0xc68a</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae0868a]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(+0xc9f2</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae089f2]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libglusterfs.so.0(+0x9b96f</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f146038596f]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libglusterfs.so.0(+0x9bc46</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f1460385c46]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libpthread.so.0(+0x75da</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145f0d15da]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libc.so.6(clone+0x3f</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145e9a7eaf]</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">callstack[2432] expires. count=1 size=104/104 alloc=362 free=0<br>
/home/robot/libleak/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libleak.so(malloc+0x25</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f1460604065]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(CRYPTO_malloc+0x58</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ecd9978]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(BN_MONT_CTX_new+0x17</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed48627]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(BN_MONT_CTX_set_locked+0x6d</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ed489fd]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(+0xff4d9</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed6a4d9]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(int_rsa_verify+0x1cd</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed6d41d]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(RSA_verify+0x32</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed6d972]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(+0x107ff5</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed72ff5]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libcrypto.so.10(EVP_VerifyFinal+0x211</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145ed9dd51]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libssl.so.10(ssl3_get_cert_verify+0x5bb</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145abac06b]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libssl.so.10(</span></u><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:red">ssl3_accept</span></u><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">+0x988</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145abad948]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(ssl_complete_connection+0x5e</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae00f3a]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(+0xc16d</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae0816d]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(+0xc68a</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae0868a]<br>
/usr/lib64/glusterfs/3.12.15/rpc-transport/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">socket.so(+0xc9f2</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">)
[0x7f145ae089f2]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libglusterfs.so.0(+0x9b96f</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f146038596f]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libglusterfs.so.0(+0x9bc46</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f1460385c46]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libpthread.so.0(+0x75da</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145f0d15da]<br>
/lib64/</span><u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:blue">libc.so.6(clone+0x3f</span></u><span lang="EN-US" style="font-size:10.0pt;font-family:"Segoe UI",sans-serif;color:#1A1A1A">) [0x7f145e9a7eaf]</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:14.0pt;font-family:DengXian">one interesting thing is that the memory goes up to about 300m then it stopped increasing !!!</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:14.0pt;font-family:DengXian">I am wondering if this is caused by open-ssl library? But when I search from openssl community, there is no such
issue reported before.</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:14.0pt;font-family:DengXian">Is glusterfs using ssl_accept correctly?</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">cynthia</span><span lang="EN-US"><o:p></o:p></span></p>
<div>
<div style="border:none;border-top:solid windowtext 1.0pt;padding:3.0pt 0cm 0cm 0cm;border-color:currentcolor">
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Zhou,
Cynthia (NSB - CN/Hangzhou) <br>
<b>Sent:</b> Monday, May 06, 2019 10:34 AM<br>
<b>To:</b> 'Amar Tumballi Suryanarayan' <<a href="mailto:atumball@redhat.com" target="_blank">atumball@redhat.com</a>><br>
<b>Cc:</b> Milind Changire <<a href="mailto:mchangir@redhat.com" target="_blank">mchangir@redhat.com</a>>;
<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a><br>
<b>Subject:</b> RE: [Gluster-devel] glusterfsd memory leak issue found after enable ssl</span><span lang="EN-US"><o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Hi,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Sorry, I am so busy with other issues these days, could you help me to submit my patch for review? It is based on
glusterfs3.12.15 code. But even with this patch , memory leak still exists, from memory leak tool it should be related with ssl_accept, not sure if it is because of openssl library or because improper use of ssl interfaces.</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">--- a/rpc/rpc-transport/socket/src/socket.c</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+++ b/rpc/rpc-transport/socket/src/socket.c</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">@@ -1019,7 +1019,16 @@ static void __socket_reset(rpc_transport_t *this) {</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> memset(&priv->incoming, 0, sizeof(priv->incoming));</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> event_unregister_close(this->ctx->event_pool, priv->sock, priv->idx);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">-</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ if(priv->use_ssl&& priv->ssl_ssl)</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ {</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ gf_log(this->name, GF_LOG_INFO,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ "clear and reset for socket(%d), free ssl ",</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ priv->sock);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ SSL_shutdown(priv->ssl_ssl);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ SSL_clear(priv->ssl_ssl);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ SSL_free(priv->ssl_ssl);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ priv->ssl_ssl = NULL;</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ }</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> priv->sock = -1;</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> priv->idx = -1;</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> priv->connected = -1;</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">@@ -4238,6 +4250,16 @@ void fini(rpc_transport_t *this) {</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> pthread_mutex_destroy(&priv->out_lock);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> pthread_mutex_destroy(&priv->cond_lock);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> pthread_cond_destroy(&priv->cond);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ if(priv->use_ssl&& priv->ssl_ssl)</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ {</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ gf_log(this->name, GF_LOG_INFO,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ "clear and reset for socket(%d), free ssl ",</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ priv->sock);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ SSL_shutdown(priv->ssl_ssl);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ SSL_clear(priv->ssl_ssl);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ SSL_free(priv->ssl_ssl);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ priv->ssl_ssl = NULL;</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">+ }</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> if (priv->ssl_private_key) {</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> GF_FREE(priv->ssl_private_key);</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> }</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Amar
Tumballi Suryanarayan <<a href="mailto:atumball@redhat.com" target="_blank">atumball@redhat.com</a>>
<br>
<b>Sent:</b> Wednesday, May 01, 2019 8:43 PM<br>
<b>To:</b> Zhou, Cynthia (NSB - CN/Hangzhou) <<a href="mailto:cynthia.zhou@nokia-sbell.com" target="_blank">cynthia.zhou@nokia-sbell.com</a>><br>
<b>Cc:</b> Milind Changire <<a href="mailto:mchangir@redhat.com" target="_blank">mchangir@redhat.com</a>>;
<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a><br>
<b>Subject:</b> Re: [Gluster-devel] glusterfsd memory leak issue found after enable ssl</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">Hi Cynthia Zhou,<o:p></o:p></span></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">Can you post the patch which fixes the issue of missing free? We will continue to investigate the leak further, but would really appreciate getting the patch
which is already worked on land into upstream master.<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">-Amar<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">On Mon, Apr 22, 2019 at 1:38 PM Zhou, Cynthia (NSB - CN/Hangzhou) <<a href="mailto:cynthia.zhou@nokia-sbell.com" target="_blank">cynthia.zhou@nokia-sbell.com</a>>
wrote:<o:p></o:p></span></p>
</div>
<blockquote style="border:none;border-left:solid windowtext 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0cm;margin-bottom:5.0pt;border-color:currentcolor currentcolor currentcolor rgb(204,204,204)">
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Ok, I am clear now.</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">I</span><span style="font-size:10.5pt;font-family:DengXian">’<span lang="EN-US">ve added ssl_free in socket reset
and socket finish function, though glusterfsd memory leak is not that much, still it is leaking, from source code I can not find anything else,</span></span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Could you help to check if this issue exists in your env? If not I may have a try to merge your patch .</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">Step</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="gmail-m3980449368408844904gmail-m-1109814905353783812gmail-m3903278274190134145msolistparagraph" style="margin-left:18.0pt">
<span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">1></span><span lang="EN-US" style="font-size:7.0pt;font-family:"Times New Roman",serif">
</span><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">while true;do gluster v heal <vol-name> info,</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="gmail-m3980449368408844904gmail-m-1109814905353783812gmail-m3903278274190134145msolistparagraph" style="margin-left:18.0pt">
<span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">2></span><span lang="EN-US" style="font-size:7.0pt;font-family:"Times New Roman",serif">
</span><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">check the vol-name glusterfsd memory usage, it is obviously increasing.</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian">cynthia</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US" style="font-size:10.5pt;font-family:DengXian"> </span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Milind
Changire <<a href="mailto:mchangir@redhat.com" target="_blank">mchangir@redhat.com</a>>
<br>
<b>Sent:</b> Monday, April 22, 2019 2:36 PM<br>
<b>To:</b> Zhou, Cynthia (NSB - CN/Hangzhou) <<a href="mailto:cynthia.zhou@nokia-sbell.com" target="_blank">cynthia.zhou@nokia-sbell.com</a>><br>
<b>Cc:</b> Atin Mukherjee <<a href="mailto:amukherj@redhat.com" target="_blank">amukherj@redhat.com</a>>;
<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a><br>
<b>Subject:</b> Re: [Gluster-devel] glusterfsd memory leak issue found after enable ssl</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
<div>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">According to BIO_new_socket() man page ...<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span lang="EN-US" style="background:#FFF2CC">If the close flag is set then the socket is shut down and closed when the BIO is freed.</span></b><span lang="EN-US"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">For Gluster to have more control over the socket shutdown, the BIO_NOCLOSE flag is set. Otherwise, SSL takes control of socket shutdown whenever BIO is freed.<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
</div>
</div>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="https://lists.gluster.org/mailman/listinfo/gluster-devel" target="_blank">https://lists.gluster.org/mailman/listinfo/gluster-devel</a><o:p></o:p></span></p>
</blockquote>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"><br clear="all">
<o:p></o:p></span></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US"> <o:p></o:p></span></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">--
<o:p></o:p></span></p>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span lang="EN-US">Amar Tumballi (amarts)<o:p></o:p></span></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;margin-bottom:12.0pt"><span lang="EN-US">_______________________________________________<br>
<br>
Community Meeting Calendar:<br>
<br>
APAC Schedule -<br>
Every 2nd and 4th Tuesday at 11:30 AM IST<br>
Bridge: <a href="https://bluejeans.com/836554017" target="_blank">https://bluejeans.com/836554017</a><br>
<br>
NA/EMEA Schedule -<br>
Every 1st and 3rd Tuesday at 01:00 PM EDT<br>
Bridge: <a href="https://bluejeans.com/486278655" target="_blank">https://bluejeans.com/486278655</a><br>
<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="https://lists.gluster.org/mailman/listinfo/gluster-devel" target="_blank">https://lists.gluster.org/mailman/listinfo/gluster-devel</a><o:p></o:p></span></p>
</blockquote>
</div>
</div>
</div>
</div>
</blockquote>
</div>
</div>
</body>
</html>