On (11/04/13 10:29), Jakub Hrozek wrote:
On Thu, Apr 11, 2013 at 10:28:01AM +0200, Jakub Hrozek wrote:
> Just one nitpick, can you change this block:
>
> + primary_file = talloc_strdup_append(discard_const_p(char, dir),
> + "/primary");
> + if (!primary_file) {
> + DEBUG(SSSDBG_CRIT_FAILURE, ("talloc_strdup_append failed.\n"));
> + ret = ENOMEM;
> + goto done;
> + }
>
> Change dir to be "char *" not "const char*" and use
> talloc_asprintf(tmp_ctx, "%s/%s", dir);
talloc_asprintf(tmp_ctx, "%s/primary", dir);
sorry.
Patch attached.
LS