shutter-network / rolling-shutter

Rolling Shutter is an MEV protection system to be plugged into rollups.
https://twitter.com/project_shutter/
28 stars 10 forks source link

tx-pointer gets overridden by any key(share) message #498

Closed fredo closed 3 months ago

fredo commented 4 months ago

@konradkonrad please post your observations

konradkonrad commented 3 months ago

I was mistaken:

It looked as if a rogue keyper could arbitrarily change other keyper's tx-pointers:

https://github.com/shutter-network/rolling-shutter/blob/f21a5db5b27894dddc384d975418397c8858832e/rolling-shutter/keyperimpl/gnosis/handlers.go#L295-L315

as long as the message validates for the receivers.

However, the prerequisites for validation are:

So this is "threshold safe", as far as I understand!

The tx-pointer value that will be updated by a DecryptionKeys message, is being voted upon during KeyShares gossiping.