krb5 commit: Follow-up on krb5_gss_duplicate_name() null check
Greg Hudson <[email protected]>
| Newsgroups | gmane.comp.encryption.kerberos.cvs |
|---|---|
| Message-ID | <[email protected]> |
https://github.com/krb5/krb5/commit/e8a932e2bdf822c655fa8faebd23f3f0fd756dab commit e8a932e2bdf822c655fa8faebd23f3f0fd756dab Author: Greg Hudson <[email protected]> Date: Tue Sep 24 01:47:18 2019 -0400 Follow-up on krb5_gss_duplicate_name() null check Commit 9fd7bc179f0bd74fc83c1edf0247dcfd87fc73e6 missed a second null check, so the forward-null defect was not squashed. Remove that one as well. src/lib/gssapi/krb5/duplicate_name.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/src/lib/gssapi/krb5/duplicate_name.c b/src/lib/gssapi/krb5/duplicate_name.c index ea53e9c..670b2b5 100644 --- a/src/lib/gssapi/krb5/duplicate_name.c +++ b/src/lib/gssapi/krb5/duplicate_name.c @@ -38,8 +38,7 @@ krb5_gss_duplicate_name(OM_uint32 *minor_status, const gss_name_t input_name, code = krb5_gss_init_context(&context); if (code) { - if (minor_status) - *minor_status = code; + *minor_status = code; return GSS_S_FAILURE; }