crytic / not-so-smart-contracts

Examples of Solidity security issues

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Transaction Ordering Vuln

ESultanik opened this issue · comments

I think this is now addressed by race_condition

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 100.0 DAI (100.0 USD @ $1.0/DAI) attached to it as part of the Ethereum Community Fund via ECF Web 3.0 Infrastructure Fund fund__.__

What are you looking for?

@salkuyateh93 the test specifies

"Add an example like this: https://github.com/ConsenSys/smart-contract-best-practices#transaction-ordering-dependence"

And there is no example under the link

The link doesn't work anymore because it linked to a README that has been changed.

A correct link for that could be e.g. https://github.com/ConsenSys/smart-contract-best-practices/tree/0b67609241f8a56aa50ff3fd9af1bf1e5b546149#transaction-ordering-dependence

(I found a commit around the date of this issue and clicked on browse files).

Anyway, as @mgcolburn pointed out this is addressed by race condition so I am closing this issue.

@dguido should this issue have funding for a Gitcoin bounty or should I cancel this bounty? Thanks! 👍

Whoops, I think cancel it.

Sure bet, cancelling it :)

Issue Status: 1. Open 2. Cancelled


The funding of 100.0 DAI (100.0 USD @ $1.0/DAI) attached to this issue has been cancelled by the bounty submitter