zips icon indicating copy to clipboard operation
zips copied to clipboard

[protocol spec] Section 5.4.1.9 (Sinsemilla Hash Function) declares use of LEOS2IP when it should declare use of LEBS2IP

Open oxarbitrage opened this issue 1 year ago • 1 comments

I might be wrong but it seems to be an error with some functions defined and later used in the Sinsemilla hash function section of the zcash protocol.

The Sinsemilla hash function specified defines the next 2 functions to be used as part of the algorithm:

sinsemilla1

However:

  • I2LEOSP is used in the S function, this is ok:

sinsemilla1 5

  • But the next and only use of another function of this kind is in the pad function:

sinsemilla2

So LEOStoIP is not used and LEBStoIP is used instead, which is not defined.

This is a normal oversight as the names are very long and similar but also confusing:

If pad is correct and LEBStoIP is the one that is going to be used then we should replace LEOStoIP in the definition with LEBStoIP. Otherwise we should change pad to use LEOStoIP

oxarbitrage avatar May 06 '24 17:05 oxarbitrage

You're correct, the reference should be to LEBS2IP. I'll fix it.

daira avatar May 12 '24 01:05 daira