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;
     }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.