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

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

krb5 commit: Initialize status in krb5_ldap_parse_db_params

daemon@ATHENA.MIT.EDU (Greg Hudson)
Mon Mar 11 16:14:46 2013

Date: Mon, 11 Mar 2013 16:14:41 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-Id: <201303112014.r2BKEffk017608@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

https://github.com/krb5/krb5/commit/233e97338784196ac610aaaffc74d4e7ba03ca9c
commit 233e97338784196ac610aaaffc74d4e7ba03ca9c
Author: Greg Hudson <ghudson@mit.edu>
Date:   Mon Mar 11 16:09:06 2013 -0400

    Initialize status in krb5_ldap_parse_db_params
    
    If db_args is non-null but empty, status could be returned without
    being initialized; gcc with optimization correctly warns about this,
    causing a build failure.  (This bug was introduced by
    0b1dc2f93da4c860dd27f1ac997617b712dff383 which was pushed after the
    1.11 release branch, so it isn't in any release.)

 src/plugins/kdb/ldap/libkdb_ldap/ldap_misc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/plugins/kdb/ldap/libkdb_ldap/ldap_misc.c b/src/plugins/kdb/ldap/libkdb_ldap/ldap_misc.c
index c4024b8..7659b9b 100644
--- a/src/plugins/kdb/ldap/libkdb_ldap/ldap_misc.c
+++ b/src/plugins/kdb/ldap/libkdb_ldap/ldap_misc.c
@@ -233,7 +233,7 @@ krb5_error_code
 krb5_ldap_parse_db_params(krb5_context context, char **db_args)
 {
     char *opt = NULL, *val = NULL;
-    krb5_error_code status;
+    krb5_error_code status = 0;
     krb5_ldap_context *lctx = context->dal_handle->db_context;
 
     if (db_args == NULL)
_______________________________________________
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