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

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

krb5 commit [krb5-1.9]: Fix condition with empty body

daemon@ATHENA.MIT.EDU (Tom Yu)
Tue Apr 23 23:08:46 2013

Date: Tue, 23 Apr 2013 23:08:38 -0400
From: Tom Yu <tlyu@mit.edu>
Message-Id: <201304240308.r3O38c5c030306@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/8dfe94281249a1b16ec725be2ade382092168b82
commit 8dfe94281249a1b16ec725be2ade382092168b82
Author: David Benjamin <davidben@mit.edu>
Date:   Wed Mar 13 01:08:01 2013 -0400

    Fix condition with empty body
    
    Found by clang's warnings.
    
    (cherry picked from commit 18796a2eb6c05706c6635453b3e425955aab93f4)
    
    (cherry picked from commit 63b5902e83eb7d47f2eb061acde2566ec1d45bc8)
    
    ticket: 7623 (new)
    version_fixed: 1.9.5
    status: resolved

 src/lib/kdb/keytab.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/kdb/keytab.c b/src/lib/kdb/keytab.c
index dbbbe75..76feacc 100644
--- a/src/lib/kdb/keytab.c
+++ b/src/lib/kdb/keytab.c
@@ -42,8 +42,8 @@ static krb5_error_code
 krb5_ktkdb_get_name(krb5_context context, krb5_keytab keytab,
                     char *name, unsigned int namelen)
 {
-    if (strlcpy(name, "KDB:", namelen) >= namelen);
-    return KRB5_KT_NAME_TOOLONG;
+    if (strlcpy(name, "KDB:", namelen) >= namelen)
+        return KRB5_KT_NAME_TOOLONG;
     return 0;
 }
 
_______________________________________________
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