View Single Post
  #7  
Old 08-22-2019, 03:30
chants chants is offline
VIP
 
Join Date: Jul 2016
Posts: 723
Rept. Given: 35
Rept. Rcvd 48 Times in 30 Posts
Thanks Given: 665
Thanks Rcvd at 1,050 Times in 475 Posts
chants Reputation: 48
@debugasm I would really hope to hear the results needed for the change. Especially try:
Quote:
https://github.com/hashcat/hashcat/blob/master/OpenCL/inc_types.h
Code:
typedef struct pw
{
  u32 i[2148];
  u32 pw_len;
} pw_t;
Quote:
https://github.com/hashcat/hashcat/blob/master/OpenCL/inc_rp.h
https://github.com/hashcat/hashcat/blob/master/include/rp.h
Code:
#define RP_PASSWORD_SIZE  8192
I suspect these are enough (the RP_PASSWORD_SIZE maybe not even needed here as I think there are no rules in use), that it will work and not crash.

The problem is the authors did not want to use dynamic memory for any size - since this would slow down the project significantly. And to fix a huge buffer size like this would negatively impact a lot of copy and other operations making them slower significantly. So how exactly to fix this is an interesting question. Instead they hardcoded the size values, knowing but not caring that there is correlation between them, and forcing custom compilation for now. I think the optimized code needs to be built on the fly which is a somewhat tricky thing to achieve though in the future I expect to see this become a regular thing.
Reply With Quote
The Following User Says Thank You to chants For This Useful Post:
p4r4d0x (08-22-2019)