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

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

krb5 commit: Fix argment validation in krb5_make_fulladdr

daemon@ATHENA.MIT.EDU (Greg Hudson)
Sat Jul 12 15:19:39 2014

Date: Sat, 12 Jul 2014 15:19:26 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-Id: <201407121919.s6CJJQxd021475@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/347e70a0649d976f10236d682ee4999569b964d4
commit 347e70a0649d976f10236d682ee4999569b964d4
Author: Lukas Slebodnik <lslebodn@redhat.com>
Date:   Sat Jun 21 16:11:31 2014 +0200

    Fix argment validation in krb5_make_fulladdr
    
    Check kaddr and kport, not kport twice.
    
    [ghudson@mit.edu: clarified commit message, minor style change]

 src/lib/krb5/os/mk_faddr.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/lib/krb5/os/mk_faddr.c b/src/lib/krb5/os/mk_faddr.c
index 7e125cf..2577df3 100644
--- a/src/lib/krb5/os/mk_faddr.c
+++ b/src/lib/krb5/os/mk_faddr.c
@@ -42,7 +42,7 @@ krb5_make_fulladdr(krb5_context context, krb5_address *kaddr,
     krb5_int32 tmp32;
     krb5_int16 tmp16;
 
-    if ((kport == NULL) || (kport == NULL))
+    if (kaddr == NULL || kport == NULL)
         return EINVAL;
 
     raddr->length = kaddr->length + kport->length + (4 * sizeof(krb5_int32));
_______________________________________________
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