Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 17 Sep 2015 23:04:54 +0200
From: magnum <>
Subject: Re: "struct db_password" allocation

On 2015-09-17 22:30, Solar Designer wrote:
> On Thu, Sep 17, 2015 at 11:26:18PM +0300, Solar Designer wrote:
>> I just found that someone added a field called "uid" to "struct
>> db_password"
> BTW, this looks wasteful even with the memory allocation corrected.
> The only use is:
>                  log_guess(crk_db->options->flags & DB_LOGIN ? replogin : "?",
>                            crk_db->options->flags & DB_LOGIN ? repuid : "",
> Maybe we should drop this field, or maybe we should merge its value into
> the login string at loading instead of maintaining a separate pointer.
> Even if we don't want to fully merge them, we can do smarter - e.g.,
> treat login as a "double string" - assume that there's a second string
> after the first NUL unless the first string was empty.  Or something.

It's main use (or at least the reason Jim implemented it) is to get 
BSSID printed/logged when cracking WPA-PSK. In that case, the ESSID is 
the "login" and sometimes there are many same ESSID so we need to see 
the BSSID as well (but we do not want it used by Single mode so it can't 
be included in login). There might be other uses too. I'll look into 
that double string idea!


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.