changes done. Please check the patch.


On Mon, Apr 22, 2013 at 12:55 PM, Jakub Hrozek <jhrozek@redhat.com> wrote:
On Tue, Apr 16, 2013 at 07:27:20PM +0530, Abhishek Singh wrote:
> Changes done. Please check the patch.
>
> thanks,
>

Hi Abishek

let's do one last change before I push the patch -- test the strdup
output for being non-NULL and then free the duplicate when it's not
needed:

    basec = strdup(get_filepath(path));
+   assert_non_null(basec);
    relativepath = basename(basec);
    fd = sss_openat_cloexec(dir_fd, relativepath, flags, &ret);
+   free(basec)


Thanks for the patch!