On 12/19/2012 04:54 PM, Ondrej Kos wrote:
On 19/12/12 12:45, Pavel Březina wrote:
> On 12/19/2012 10:27 AM, Ondrej Kos wrote:
>> i noticed warning during compilation:
>>
>> src/providers/ldap/sdap_sudo.c: In function
>> 'sdap_sudo_schedule_refresh':
>> src/providers/ldap/sdap_sudo.c:1236:9: warning: 'send_fn' may be used
>> uninitialized in this function [-Wmaybe-uninitialized]
>>
>> patch is attached
>>
>> Ondra
>
> Hi,
> thanks. FYI you need at least -O1.
>
> I see also:
> src/providers/ldap/sdap_async_sudo.c: In function
> 'sdap_sudo_load_sudoers_done':
> src/providers/ldap/sdap_async_sudo.c:579:9: warning: 'ret' may be used
> uninitialized in this function [-Wmaybe-uninitialized]
> src/providers/ldap/sdap_async_sudo.c:543:13: note: 'ret' was declared
> here
>
> Can you fix it too in this patch?
> _______________________________________________
> sssd-devel mailing list
> sssd-devel(a)lists.fedorahosted.org
>
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
thanks, i forgot to raise optimalization level after some debugging
new patch is attached
Ondra
Hi,
nack. We don't want to initialize ret = EOK. We should set it after the
for loop instead. But this touches the behaviour of this function so
I'll send it as another patch.
Ack to the *original* patch.