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