<div dir="ltr"><div>Hello Andreas,<br><br></div>Thanks for your response. Please see my response inline.<br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 24, 2014 at 2:29 PM, Andreas Steffen <span dir="ltr"><<a href="mailto:andreas.steffen@strongswan.org" target="_blank">andreas.steffen@strongswan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Avesh,<br>
<br>
the chunk_create() function does not allocate any dynamic memory.<br>
It just wraps a pointer and a length into a static chunk_t object.<br>
<br></blockquote><div><br>Then its fine.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
 identity_req = chunk_create(IdentityReq, IdentityReqLen);<br></blockquote><div><br></div><div>In any case as per my understanding, the above statement should be called inside the "if (idreq_filename)" block.<br>
 <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
THE IdentityReq structure is made available by the Trousers command<br>
<br>
result = Tspi_TPM_CollateIdentityRequest(hTPM, hSRK, hPCAKey, 0, NULL,<br>
                                        hIdentKey, TSS_ALG_AES, &IdentityReqLen, &IdentityReq);<br>
<br>
At the end of aikgen I'm freeing the complete TSS content and any<br>
internal memory storing IdentityReq will be released.</blockquote><div><br></div><div>If I understand you correctly, then it means that the call to Tspi_Context_FreeMemory in exit_aikgen is freeing IdentityReq. Then it seems fine to me.<br>
<br></div><div>Thanks and Regards<br>Avesh<br></div><div><br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> Using our<br>
leak_detective I noticed only one small memory leak in the libtspi<br>
library. If I find the time I'm going to locate the leak and post a<br>
patch to the Trousers project.<br></blockquote><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Best regards<br>
<br>
Andreas<br>
<div><div class="h5"><br>
On 05/23/2014 09:34 PM, Avesh Agarwal wrote:<br>
> Hi,<br>
><br>
> I was looking at aikgen.c implementation, and I found that that idreq<br>
> buffer is being created even without idreq option, it seems to be<br>
> leading to memory leaks. I have created a patch to address it.<br>
><br>
> Thanks and Regards<br>
> Avesh<br>
<br>
</div></div>======================================================================<br>
Andreas Steffen                         <a href="mailto:andreas.steffen@strongswan.org">andreas.steffen@strongswan.org</a><br>
strongSwan - the Open Source VPN Solution!          <a href="http://www.strongswan.org" target="_blank">www.strongswan.org</a><br>
Institute for Internet Technologies and Applications<br>
University of Applied Sciences Rapperswil<br>
CH-8640 Rapperswil (Switzerland)<br>
===========================================================[ITA-HSR]==<br>
<br>
</blockquote></div><br></div></div>