Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANO7a6w=hgdC9YUr4kpfms-k8nBELFa_O8RvY85=qMbDzsfkHg@mail.gmail.com>
Date: Sat, 8 Sep 2012 18:36:00 +0530
From: Dhiru Kholia <dhiru.kholia@...il.com>
To: john-dev@...ts.openwall.com
Subject: Re: New formats: KRB5-18 and KRB5-23

On Sat, Sep 8, 2012 at 6:21 PM, Dhiru Kholia <dhiru.kholia@...il.com> wrote:
> On Sat, Sep 8, 2012 at 6:17 PM, magnum <john.magnum@...hmail.com> wrote:
>> That looks correct to me. Is there something similar in krb5-23?
>
> I will take a look today.

Patch for krb5-23. Linux OMP works fine now.

diff --git a/src/krb5-23_fmt.c b/src/krb5-23_fmt.c
index c0d239d..ab03d03 100644
--- a/src/krb5-23_fmt.c
+++ b/src/krb5-23_fmt.c
@@ -66,9 +66,6 @@ static struct fmt_tests kinit_tests[] = {
 static char (*saved_key)[PLAINTEXT_LENGTH + 1];
 static ARCH_WORD_32 (*crypt_out)[8];

-static krb5_error_code ret;
-static krb5_data string;
-static krb5_keyblock key;
 static krb5_data salt;
 static krb5_enctype enctype;

@@ -158,16 +155,18 @@ static void crypt_all(int count)
   for (index = 0; index < count; index++)
 #endif
     {
+      krb5_data string;
+      krb5_keyblock key;

       string.data = saved_key[index];
       string.length = strlen(saved_key[index]);
-      ret = krb5_c_string_to_key_with_params(NULL,
+      krb5_c_string_to_key_with_params(NULL,
                                             enctype,
                                             &string,
                                             &salt,
                                             NULL,
                                             &key);
-      for(; i < key.length / 4; i++){
+      for(i=0; i < key.length / 4; i++){
              crypt_out[index][i] = (key.contents[4 * i]) |
                      (key.contents[4 * i + 1] << 8) |
                      (key.contents[4 * i + 2] << 16) |


The problem was that 'i' was not getting reset to 0.

-- 
Cheers,
Dhiru

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.