raiden-network / raiden-contracts

Raiden Network Smart Contracts
MIT License
53 stars 45 forks source link

Make balance_hash less predictable #1121

Open pirapira opened 5 years ago

pirapira commented 5 years ago

balance_hash's are meant to hide concrete transferred amounts from MonitoringServices, but in the current form, they have not enough entropy so that an observer can easily recover the contents of balance hashes from monitoring requests.

This issue keeps track of adding a salt to balance_hash.

pirapira commented 5 years ago

A native implementation causes

data/source/raiden/TokenNetwork.sol:735:9: Compiler error: Stack too deep, try removing local variables.