krb5 commit: Follow-up on krb5_gss_duplicate_name() null check

Greg Hudson ghudson at mit.edu
Tue Sep 24 02:01:17 EDT 2019


https://github.com/krb5/krb5/commit/e8a932e2bdf822c655fa8faebd23f3f0fd756dab
commit e8a932e2bdf822c655fa8faebd23f3f0fd756dab
Author: Greg Hudson <ghudson at mit.edu>
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;
     }
 


More information about the cvs-krb5 mailing list