cipher: Fix regression in Padlock support.
authorWerner Koch <wk@gnupg.org>
Thu, 18 Apr 2013 12:40:43 +0000 (14:40 +0200)
committerWerner Koch <wk@gnupg.org>
Thu, 18 Apr 2013 14:11:49 +0000 (16:11 +0200)
* cipher/rijndael.c (do_setkey): Remove dummy padlock key generation case
and use the standard one.
--

This is really a brown paper bag bug.  I should have been able to
fix it by a bit of code staring or bi-secting it myself.  Instead
Rafaël Carré did this and with the donation of a VIA nano board from
Stefan Krüger.  Thanks to both of you.

(regression since commit b825c5db17292988d261fefdc83cbc43d97d4b02)

Signed-off-by: Werner Koch <wk@gnupg.org>
(cherry picked from commit f1f016855418aae561ede4472590d45a24ab4476)

THANKS
cipher/rijndael.c

diff --git a/THANKS b/THANKS
index 41f4c77..0f47d4f 100644 (file)
--- a/THANKS
+++ b/THANKS
@@ -130,6 +130,7 @@ Stephan Müller             smueller at atsec com
 Stephane Corthesy          stephane@sente.ch
 Stefan Karrmann           S.Karrmann@gmx.net
 Stefan Keller             dres@cs.tu-berlin.de
+Stefan Krüger              stadtkind2 at gmx de
 Steffen Ullrich           ccrlphr@xensei.com
 Steffen Zahn              zahn@berlin.snafu.de
 Steven Bakker             steven@icoe.att.com
index a3080f5..8d4036b 100644 (file)
@@ -285,15 +285,10 @@ do_setkey (RIJNDAEL_context *ctx, const byte *key, const unsigned keylen)
 
   ctx->rounds = rounds;
 
+  /* NB: We don't yet support Padlock hardware key generation.  */
+
   if (0)
     ;
-#ifdef USE_PADLOCK
-  else if (ctx->use_padlock)
-    {
-      /* Nothing to do as we support only hardware key generation for
-         now.  */
-    }
-#endif /*USE_PADLOCK*/
 #ifdef USE_AESNI_is_disabled_here
   else if (ctx->use_aesni && ctx->rounds == 10)
     {