feat: MIMC security considerations

[Soleimani193]

Description of the Problem In the Original MIMC paper, the authors apply Sponge structure over MIMC permutation, while the gnark implementation uses Miyaguchi-Preneel structure.

Miyaguchi-Preneel is vulnerable to the length extension attack and should not be used for applications like MAC (where one hides a secret key inside the hash).

The positive side is a possible efficiency gain for the applications that are irrelevant to the length extension attack (e.g., snark) . Here is an estimation (footnote 19).

Suggestions/Solutions

1. The user should be warned about the use of MIMC hash based on Miyaguchi-Preneel structure.
2. The implementation of MIMC based on Sponge structure would be very helpful
3. A benchmark that can probably assert the efficiency gain of Miyaguchi-Preneel for snark applications.

[giuliop]

In their FAQ the MiMC authors comment against using a Miyaguchi-Preneel structure and add that if going that route they would recommend increasing the number of rounds. The number of rounds in gnark's implementation matches the original paper and does not follow the recommendation to increase it with a Miyaguchi-Preneel structure.

What is the line of thinking followed to make these choices in gnark?

[ivokub]

I am not sure, I haven't been involved in the implementation, but I would assume the main reasoning would have been to have compatibility with alternative libraries. Length-extension attacks are somewhat mitigated as MiMC is mostly used for in-circuit hashing (and we have the native version in gnark-crypto for compatibility), where the input lengths are fixed.

But I do agree that the issue and I think it needs some consideration. There are several parallel lines of work regarding algebraic hashing:

• adding interfaces which are more natural to algebraic hashing https://github.com/Consensys/gnark-crypto/issues/448
• adding Poseidon hash https://github.com/Consensys/gnark-crypto/issues/494
• implementing SAFE API https://github.com/Consensys/gnark/issues/1122
• MiMC parametrization https://github.com/Consensys/gnark-crypto/issues/221
• etc

I would definitely add the points raised by @Soleimani193:

• The user should be warned about the use of MIMC hash based on Miyaguchi-Preneel structure.
• The implementation of MIMC based on Sponge structure would be very helpful