Skip to content

Latest commit

 

History

History
35 lines (25 loc) · 1.64 KB

Unpredictable behavior for users due to admin front running or general bad timing.md

File metadata and controls

35 lines (25 loc) · 1.64 KB

In a number of cases, administrators of contracts can update or upgrade things in the system without warning.

This has the potential to violate a security goal of the system.

Specifically, privileged roles could use front running to make malicious changes just ahead of incoming transactions, or purely accidental negative effects could occur due to the unfortunate timing of changes.

In general users of the system should have assurances about the behavior of the action they're about to take.

Recommendation:

We recommend giving the user advance notice of changes with a time lock.

For example, make all system-parameter and upgrades require two steps with a mandatory time window between them.

The first step merely broadcasts to users that a particular change is coming, and the second step commits that change after a suitable waiting period.

This allows users that do not accept the change to withdraw immediately.


Slide Screenshot

118.jpg


Slide Text

  • ConsenSys Audit 1inch Finding 5.7
  • Privileged Roles & Timing
  • Critical Changes & Front-running
  • Two-step Change w/Time Window for Users

References


Tags