On Fri, Jan 09, 2015 at 05:11:38PM +0100, Pavel Reichl wrote:
> On 01/09/2015 03:27 PM, Jakub Hrozek wrote:
>> On Thu, Jan 08, 2015 at 01:26:15PM +0100, Pavel Reichl wrote:
>>> On 01/07/2015 11:51 AM, Jakub Hrozek wrote:
>>>> A very simple patch, see the commit message for details.
>>>>
>>>>
>>>> _______________________________________________
>>>> sssd-devel mailing list
>>>> sssd-devel(a)lists.fedorahosted.org
>>>>
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>>> I'm sorry, but I think you might have forgotten to remove the duplicated
>>> part of code from sysdb_gpo_get_gpo_result_setting()...?
>> Yes :-)
>>
>> New patch is attached.
> Please remove (unused) variable declarations:
>
> --- a/src/db/sysdb_gpo.c
> +++ b/src/db/sysdb_gpo.c
> @@ -591,8 +591,6 @@ sysdb_gpo_get_gpo_result_setting(TALLOC_CTX *mem_ctx,
> const char **_ini_value)
> {
> errno_t ret;
> - int lret;
> - struct ldb_dn *base_dn;
> TALLOC_CTX *tmp_ctx;
> struct ldb_result *res;
> const char *ini_value;
thanks.. third time's the charm?
new patch is attached.
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel