Wrong size for memset()s in combine_keys()
authorKen Hornstein <kenh@cmf.nrl.navy.mil>
Thu, 14 Nov 2002 15:48:36 +0000 (15:48 +0000)
committerKen Hornstein <kenh@cmf.nrl.navy.mil>
Thu, 14 Nov 2002 15:48:36 +0000 (15:48 +0000)
The wrong size was used in the final memset()'s of various combine_key
temporary variables.  Pointed out by Ken Renard.

ticket: new

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@14997 dc483132-0cff-0310-8789-dd5450dbe970

src/lib/crypto/combine_keys.c

index 3bf4c2b2de2a94770fe127e0ab4a3185c73632d4..6466a95f30cdeddfc2e716b4e56e9a3d6523b249 100644 (file)
@@ -242,10 +242,10 @@ krb5_error_code krb5int_c_combine_keys
     ret = 0;
 
 cleanup:
-    memset(r1, 0, keylength);
-    memset(r2, 0, keylength);
+    memset(r1, 0, keybytes);
+    memset(r2, 0, keybytes);
     memset(rnd, 0, keybytes);
-    memset(combined, 0, keylength * 2);
+    memset(combined, 0, keybytes * 2);
     memset(output, 0, keylength);
 
     free(r1);