<div dir="ltr"><div>Hi Jakub,<br><br>&gt;You should also pass PATH_MAX-1 not PATH_MAX as the third argument of<br>
&gt;snprintf, otherwise you risk a off-by-one error in case the buffers had<br>
&gt;the same length.<br><br></div>snprintf ? Do you mean strncpy instead of instead of snprintf? since I haven&#39;t used snprintf in my code.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 9, 2013 at 3:01 AM, Jakub Hrozek <span dir="ltr">&lt;<a href="mailto:jhrozek@redhat.com" target="_blank">jhrozek@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, Apr 05, 2013 at 06:34:45PM +0530, Abhishek Singh wrote:<br>
&gt; Hi Jakub,<br>
&gt;<br>
</div><div class="im">&gt; I have made necessary changes, please review it.<br>
&gt;<br>
&gt; thanks,<br>
&gt;<br>
<br>
</div>Hi Abhishek,<br>
<br>
the test compiles and runs fine, the patches apply cleanly on top of one<br>
another and the teardown function is used cleanly.<br>
<br>
I only have one last comment -- I wasn&#39;t completely clear with how I<br>
would like to see the PATH_MAX used. PATH_MAX is a constant defined in<br>
the standard header file limits.h, you don&#39;t have to define it yourself,<br>
just include limits.h.<br>
<br>
You should also pass PATH_MAX-1 not PATH_MAX as the third argument of<br>
snprintf, otherwise you risk a off-by-one error in case the buffers had<br>
the same length.<br>
<br>
This is the last comment I have, then we&#39;ll push the patch :)<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
sssd-devel mailing list<br>
<a href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a><br>
</div></div></blockquote></div><br></div>