diff options
| author | Greg Hudson <ghudson@mit.edu> | 2022-06-03 14:30:42 -0400 |
|---|---|---|
| committer | Greg Hudson <ghudson@mit.edu> | 2022-11-05 15:56:37 -0400 |
| commit | 146e48e9a1b5ebdb30c0d57c3bd26170986371f1 (patch) | |
| tree | fb3eb8072a5be8642b3863512355f66d081c2bc6 | |
| parent | a759aae9dd50ea7ecbb1df610d963e8645b3d727 (diff) | |
| download | krb5-146e48e9a1b5ebdb30c0d57c3bd26170986371f1.zip krb5-146e48e9a1b5ebdb30c0d57c3bd26170986371f1.tar.gz krb5-146e48e9a1b5ebdb30c0d57c3bd26170986371f1.tar.bz2 | |
Fix memory leak in OTP kdcpreauth module
In otp_edata(), free the generated nonce.
(cherry picked from commit 5ad465bc8e0d957a4945218bea487b77622bf433)
ticket: 9063
version_fixed: 1.19.4
| -rw-r--r-- | src/plugins/preauth/otp/main.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/plugins/preauth/otp/main.c b/src/plugins/preauth/otp/main.c index a1b6816..2f3d7df 100644 --- a/src/plugins/preauth/otp/main.c +++ b/src/plugins/preauth/otp/main.c @@ -211,7 +211,7 @@ otp_edata(krb5_context context, krb5_kdc_req *request, krb5_pa_otp_challenge chl; krb5_pa_data *pa = NULL; krb5_error_code retval; - krb5_data *encoding; + krb5_data *encoding, nonce = empty_data(); char *config; /* Determine if otp is enabled for the user. */ @@ -239,9 +239,10 @@ otp_edata(krb5_context context, krb5_kdc_req *request, ti.iteration_count = -1; /* Generate the nonce. */ - retval = nonce_generate(context, armor_key->length, &chl.nonce); + retval = nonce_generate(context, armor_key->length, &nonce); if (retval != 0) goto out; + chl.nonce = nonce; /* Build the output pa-data. */ retval = encode_krb5_pa_otp_challenge(&chl, &encoding); @@ -258,6 +259,7 @@ otp_edata(krb5_context context, krb5_kdc_req *request, free(encoding); out: + krb5_free_data_contents(context, &nonce); (*respond)(arg, retval, pa); } |