node-scrypt icon indicating copy to clipboard operation
node-scrypt copied to clipboard

Weird hash errors with certain parameters

Open qqux opened this issue 7 years ago • 1 comments

> scrypt.kdfSync(new Buffer('asd'), s.paramsSync(5.0))
<Buffer 73 (...) 18 ... >

Seems good enough, let's try hashSync with similar parameters.

> scrypt.hashSync(new Buffer('asd'), s.paramsSync(5.0), 64, new Buffer(''))
Error: ��_
    at Object.hashSync (<redacted>/node_modules/scrypt/index.js:359:25)

Now that doesn't look right at all. Let's try the example from one of the test vectors.

> scrypt.hashSync(new Buffer(''),{"N":16,"r":1,"p":1},64,new Buffer("")).toString('hex')
'77d6 (...) 8906'

Works right.

The parameters which are generated by scrypt.paramsSync(5.0) in my case are

{ N: 18, r: 8, p: 5 }

Ubuntu Server 18.04, running node v8.11.2.

qqux avatar May 19 '18 22:05 qqux

It should be the same issue as of patch: do not reference arguments directly #120 . Does the version on @ml1nk/scrypt work?

ml1nk avatar Jun 18 '18 00:06 ml1nk