[30579] in CVS-changelog-for-Kerberos-V5

home help back first fref pref prev next nref lref last post

krb5 commit: Remove null check in krb5_gss_duplicate_name()

daemon@ATHENA.MIT.EDU (Greg Hudson)
Wed Sep 11 19:30:30 2019

Date: Wed, 11 Sep 2019 19:30:19 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-ID: <201909112330.x8BNUJva023422@drugstore.mit.edu>
To: <cvs-krb5@mit.edu>
MIME-Version: 1.0
Reply-To: krbdev@mit.edu
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu

https://github.com/krb5/krb5/commit/9fd7bc179f0bd74fc83c1edf0247dcfd87fc73e6
commit 9fd7bc179f0bd74fc83c1edf0247dcfd87fc73e6
Author: Robbie Harwood <rharwood@redhat.com>
Date:   Fri Aug 30 11:19:52 2019 -0400

    Remove null check in krb5_gss_duplicate_name()
    
    Within the krb5 mechanism, we require minor_status to be writable
    without checking.  Remove the null check in krb5_gss_duplicate_name()
    to squash a forward-null defect.

 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 b88d97d..ea53e9c 100644
--- a/src/lib/gssapi/krb5/duplicate_name.c
+++ b/src/lib/gssapi/krb5/duplicate_name.c
@@ -34,8 +34,7 @@ krb5_gss_duplicate_name(OM_uint32 *minor_status, const gss_name_t input_name,
     krb5_error_code code;
     krb5_gss_name_t princ, outprinc;
 
-    if (minor_status)
-        *minor_status = 0;
+    *minor_status = 0;
 
     code = krb5_gss_init_context(&context);
     if (code) {
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5

home help back first fref pref prev next nref lref last post