<<< Date Index >>>     <<< Thread Index >>>

Bug in Crypt::ECB perl module



There is a bug in the Crypt::ECB module that affects the way it processes 
blocks of data.  It appears that Crypt::ECB incorrectly processes the last 
block of data if it contains a singe ASCII "0".  This has been tested using the 
Blowfish, Rijndael, TripleDES, DES, and IDEA algorithms.  Below is some 
proof-of-concept code to demonstrate the problem.  The problem is produced 
where the plain-text data length is one more than ((n % 8) == 0).  I have not 
tested the Crypt::CFB or Crypt::OFB modules which I believe are based on the 
Crypt::ECB module.

Plain-text data examples:
        0
        123456780
        abcdefgh0
        12345678123456780
        ABCDEFGHabcdefgh0
        1234567812345678123456780
        ABCDEFGHabcdefgh123456780
        ... and so on...

Included below is a patch that corrects the problem in the Crypt::ECB module.  
I have attempted to contact the author, however, the email address in the 
module appears to be dead.  I have also posted this problem on one of the CPAN 
bug sites.

I don't believe there is a security vulnerability here other than mechanisms 
that use the Crypt::ECB module to encode passwords will produce incorrect 
results with specific plain-text data sets.   Um... I guess that is a security 
vulnerability in that there would be colliding passwords.

Thanks
- Bennett


Proof-of-Concept code:
   #!/usr/local/bin/perl

   use Crypt::ECB;

   my $cipher = "Blowfish";
   my $key    = "pb25YTt7d5b55711fd50bffcec4058d3e6c86bfc4c796bec2249b447";
   my $plain  = "12345678123456780";

   my $crypt = Crypt::ECB->new;

   $crypt->padding(PADDING_AUTO);
   $crypt->cipher($cipher) or die $crypt->errstring;
   $crypt->key($key);

   printf "Plain     = '%s'\n", $plain;

   my $enc = $crypt->encrypt_hex($plain);
   printf "Encrypted = '%s'\n", $enc;

   my $dec = $crypt->decrypt_hex($enc);
   printf "Decrypted = '%s'\n", $dec;


Patch:
### 
### Diff for ECB fix (output from diff -u)
###
### Test Data:
###             Plain: ILlW1nr30
###             Key:     
pb25YTt7d5b55711fd50bffcec4058d3e6c86bfc4c796bec2249b447
###             Pad:     AUTO
###
--- ECB.pm      2000-12-23 13:16:38.000000000 -0500
+++ ECB2.pm     2004-12-06 12:45:23.000000000 -0500
@@ -1,4 +1,4 @@
-package Crypt::ECB;
+package Crypt::ECB2;

 # Copyright (C) 2000  Christoph Appel, cappel@xxxxxxxxx
 #  see documentation for details
@@ -274,7 +274,7 @@
     $crypt->{Mode}   = '';
     $crypt->{buffer} = '';

-    return '' unless $data;
+    return '' unless length($data) > 0;

     my $cipher = $crypt->_getcipher;