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