[30675] in CVS-changelog-for-Kerberos-V5
krb5 commit: Fix error return case in serialization code
daemon@ATHENA.MIT.EDU (Greg Hudson)
Sun Jan 5 01:55:00 2020
Date: Sun, 5 Jan 2020 01:54:30 -0500
From: Greg Hudson <ghudson@mit.edu>
Message-ID: <202001050654.0056sUXE018853@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/82d7ffb07c81aefeabf67c7c066070efbc34400b
commit 82d7ffb07c81aefeabf67c7c066070efbc34400b
Author: Greg Hudson <ghudson@mit.edu>
Date: Mon Dec 9 11:33:57 2019 -0500
Fix error return case in serialization code
One of the changes in commit 81e47875e3de0e52fbb11d61ef30a9406497af73
unintentionally caused the return value of profile_ser_externalize()
to be ignored.
src/lib/krb5/krb/ser_ctx.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/src/lib/krb5/krb/ser_ctx.c b/src/lib/krb5/krb/ser_ctx.c
index 80cd056..9d5f9de 100644
--- a/src/lib/krb5/krb/ser_ctx.c
+++ b/src/lib/krb5/krb/ser_ctx.c
@@ -204,8 +204,11 @@ k5_externalize_context(krb5_context context,
return (kret);
/* Finally, handle profile, if appropriate */
- if (context->profile != NULL)
+ if (context->profile != NULL) {
kret = profile_ser_externalize(NULL, context->profile, &bp, &remain);
+ if (kret)
+ return (kret);
+ }
/*
* If we were successful, write trailer then update the pointer and
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5
