Openwall GNU/*/Linux - a small security-enhanced Linux distro for servers
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Fri, 25 Apr 2014 15:12:35 +0200
From: Julien Legras <julien.legras@...acktiv.com>
To: john-dev@...ts.openwall.com
Subject: Re: Compilation failure

Sorry, I missed the:

if (*regex == '=')
                 ++regex;

My patch is not working but I have no idea why the compilation is 
failing. Hope someone will figure out how to fix this.


Julien



On 25/04/2014 14:17, Julien Legras wrote:
> Hello,
>
> I pull this morning the f6c2ea0e932cf613697f5947b2418e04d9ee9be7
> revision from bleeding-jumbo branch and the compilation fails with the
> following error:
>
> gcc -c -Wall -Wdeclaration-after-statement -O2 -fomit-frame-pointer
> -I/usr/local/include          -DHAVE_CRYPT -DHAVE_DL -march=native
> -funroll-loops wordlist.c -o wordlist.o
> wordlist.c: In function ‘do_wordlist_crack’:
> wordlist.c:370:57: error: expected expression before ‘;’ token
>    regex = prepare_regex(regex, &regex_case, &regex_alpha);
>                                                           ^
>
> The regex is already modified in the prepare_regex function so there is
> no need to assign the return value. Please, tell me if I am wrong.
>
> Here is a quick fix (git diff):
>
> diff --git a/src/wordlist.c b/src/wordlist.c
> index 7eb6817..277cbf0 100644
> --- a/src/wordlist.c
> +++ b/src/wordlist.c
> @@ -367,7 +367,7 @@ void do_wordlist_crack(struct db_main *db, char
> *name, int rules)
>          if (options.activewordlistrules)
>                  log_event("- Rules: %.100s", options.activewordlistrules);
>
> -       regex = prepare_regex(regex, &regex_case, &regex_alpha);
> +       prepare_regex(regex, &regex_case, &regex_alpha);
>
>          length = db->format->params.plaintext_length;
>          if (options.force_maxlength && options.force_maxlength < length)
>
>
>
>
> Julien Legras

Powered by blists - more mailing lists

Your e-mail address:

Powered by Openwall GNU/*/Linux - Powered by OpenVZ