![]() |
|
|
|
#1
|
|||
|
|||
|
The First/Next change:... is a mistake I meant to say it is easiest to change:
Quote:
Code:
#define SALT_MAX 256 #define SALT_MAX_OLD 51 Either way I have no forward and back traced this issue - and I am guessing you changed SALT_MAX (it looks like for whatever reason PW_MAX is not used in 1450, not sure why). But you probably forgot to change the pw_t structure size. If either PW_MAX or SALT_MAX is increased about 256, the pw_t->i member needs its array size increased to max(SALT_MAX, PW_MAX)/sizeof(u32) which is currently correctly set to its corresponding 256/4=64 value. Probably why the comment "// do not try to simply change this, it will not work" is sitting right above those constants. Too bad that macro is no easy to integrate in the source due to that file being dependency free. Also please take note: Code:
#define RP_PASSWORD_SIZE 256 Quote:
Last edited by chants; 08-19-2019 at 08:31. |
|
#2
|
|||
|
|||
|
Thanks chants,
the change I made was just this: Code:
#define SALT_MAX 8192 There are other calculations, looking in the chain of routines, which should be adjusted, but you risk doing only damage. The author should have put his hands in it, but I doubt he has the time and the will to do it. I'll do some more tests. |
![]() |
| Thread Tools | |
| Display Modes | |
|
|