[26372] in CVS-changelog-for-Kerberos-V5
svn rev #25707: branches/krb5-1-10/src/lib/gssapi/krb5/
daemon@ATHENA.MIT.EDU (tlyu@mit.edu)
Tue Feb 21 23:11:56 2012
Date: Tue, 21 Feb 2012 23:11:45 -0500
From: tlyu@mit.edu
Message-Id: <201202220411.q1M4Bj55015931@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu
http://src.mit.edu/fisheye/changelog/krb5/?cs=25707
Commit By: tlyu
Log Message:
ticket: 7087
version_fixed: 1.10.1
status: resolved
Pull up r25674 from trunk
------------------------------------------------------------------------
r25674 | ghudson | 2012-02-06 18:19:08 -0500 (Mon, 06 Feb 2012) | 18 lines
ticket: 7087
status: open
target_version: 1.10.1
tags: pullup
Set display_value in krb5_gss_get_name_attribute
A backwards conditional in r25358 caused krb5_gss_get_name_attribute
not to set display_value on success. Fix the sense of the
conditional.
We still don't quite correctly handle the cases where data_to_gss()
fails, but those should be rare and the problem in those cases isn't
severe, so it can be fixed separately.
Also, value and display_value should probably be initialized to null
buffers on failure, as is common with GSS interfaces.
Changed Files:
U branches/krb5-1-10/src/lib/gssapi/krb5/naming_exts.c
Modified: branches/krb5-1-10/src/lib/gssapi/krb5/naming_exts.c
===================================================================
--- branches/krb5-1-10/src/lib/gssapi/krb5/naming_exts.c 2012-02-22 04:00:10 UTC (rev 25706)
+++ branches/krb5-1-10/src/lib/gssapi/krb5/naming_exts.c 2012-02-22 04:11:45 UTC (rev 25707)
@@ -387,7 +387,7 @@
*complete = kcomplete;
if (display_value != NULL) {
- if (code != 0)
+ if (code == 0)
code = data_to_gss(&kdisplay_value, display_value);
else
free(kdisplay_value.data);
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5
