[SSSD] [PATCH] Fix initialization of multiple variables
Ondrej Kos
okos at redhat.com
Tue Mar 12 14:03:02 UTC 2013
On 03/12/2013 02:47 PM, Michal Židek wrote:
> On 03/12/2013 01:07 PM, Ondrej Kos wrote:
>
>> diff --git a/src/providers/krb5/krb5_renew_tgt.c
>> b/src/providers/krb5/krb5_renew_tgt.c
>> index
>> e8957484286b34781f66d5dc43efdf4a3dab4530..f593edb506b4e207f6cdc786bf144fd04ea7f51c
>> 100644
>> --- a/src/providers/krb5/krb5_renew_tgt.c
>> +++ b/src/providers/krb5/krb5_renew_tgt.c
>> @@ -157,7 +157,7 @@ static errno_t renew_all_tgts(struct renew_tgt_ctx
>> *renew_tgt_ctx)
>> time_t now;
>> struct auth_data *auth_data;
>> struct renew_data *renew_data;
>> - struct tevent_timer *te;
>> + struct tevent_timer *te = NULL;
>>
>> ret = hash_entries(renew_tgt_ctx->tgt_table, &count, &entries);
>> if (ret != HASH_SUCCESS) {
>
> Could you remove the te = NULL; on line 200 in this file. It is not
> needed if the variable is initialized.
>
> Michal
Nice catch, new patch attached.
Ondra
--
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech
phone: +420-532-294-558
cell: +420-736-417-909
ext: 82-62558
loc: 1013 Brno 1 office
irc: okos @ #sssd #brno
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-initialization-of-multiple-variables.patch
Type: text/x-patch
Size: 5451 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130312/657f3a1d/attachment.bin>
More information about the sssd-devel
mailing list