summaryrefslogtreecommitdiffstats
path: root/src/lib
diff options
context:
space:
mode:
authorGreg Hudson <ghudson@mit.edu>2012-12-21 16:18:34 -0500
committerGreg Hudson <ghudson@mit.edu>2012-12-21 16:21:06 -0500
commit90e43628202157d72fb78bbb837795d91d5ce8da (patch)
tree45fa9f45f628afbd2c78c8f786afd46898abbcd9 /src/lib
parent7506becc0ac70915050e097d673e7647b99347fc (diff)
downloadkrb5-90e43628202157d72fb78bbb837795d91d5ce8da.tar.gz
krb5-90e43628202157d72fb78bbb837795d91d5ce8da.tar.xz
krb5-90e43628202157d72fb78bbb837795d91d5ce8da.zip
Initialize ret in process_pa_data
If the for loop never consults any preauth modules and must_preauth is false, we might never set ret, so we need to initialize it. The bug was introduced in 5c23bce0e8d3328bb36bc85ee10cfac486b8ae9b and is detected by some versions of gcc with -O2.
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/krb5/krb/preauth2.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/krb5/krb/preauth2.c b/src/lib/krb5/krb/preauth2.c
index a06233c27d..8d5439ccf4 100644
--- a/src/lib/krb5/krb/preauth2.c
+++ b/src/lib/krb5/krb/preauth2.c
@@ -586,7 +586,7 @@ process_pa_data(krb5_context context, krb5_get_init_creds_opt *opt,
struct krb5_preauth_context_st *pctx = context->preauth_context;
struct errinfo save = EMPTY_ERRINFO;
krb5_pa_data *pa, **pa_ptr, **mod_pa;
- krb5_error_code ret;
+ krb5_error_code ret = 0;
clpreauth_handle h;
int real, i;