On Fri, 2009-09-25 at 12:35 +0200, Jakub Hrozek wrote:
>      /* set up things like debug , signals, daemonization, etc... */
> +    debug_log_file = talloc_strdup(NULL, "sssd_pam");
> +    if (debug_log_file == NULL) return 2;
> +

debug_log_file is a process constant, it's useless to allocate memory to
copy a constant string you'll never release nor change.

Please replace with a simple:
debug_log_file = "sssd_pam";

Simo.

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to