<div dir="ltr">changes done. Please check the patch.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 22, 2013 at 12:55 PM, 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 Tue, Apr 16, 2013 at 07:27:20PM +0530, Abhishek Singh wrote:<br>
&gt; Changes done. Please check the patch.<br>
&gt;<br>
&gt; thanks,<br>
&gt;<br>
<br>
</div>Hi Abishek<br>
<br>
let&#39;s do one last change before I push the patch -- test the strdup<br>
output for being non-NULL and then free the duplicate when it&#39;s not<br>
needed:<br>
<br>
    basec = strdup(get_filepath(path));<br>
+   assert_non_null(basec);<br>
    relativepath = basename(basec);<br>
    fd = sss_openat_cloexec(dir_fd, relativepath, flags, &amp;ret);<br>
+   free(basec)<br>
<br>
<br>
Thanks for the patch!<br>
</blockquote></div><br></div>