Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong EXTCODECOPY test #235

Closed
feliam opened this issue Jul 19, 2017 · 5 comments
Closed

Wrong EXTCODECOPY test #235

feliam opened this issue Jul 19, 2017 · 5 comments

Comments

@feliam
Copy link
Contributor

feliam commented Jul 19, 2017

The following test checks for a new fffffffffffffffff account created at the end but no CREATE instruction in it. Right? (vmInputLimitsLight.json)

"12a1f52c36a8dae436b0d2d354614b5c30deb77a7d104cbf22f6b8ef8c42429c" : {
    "callcreates" : [
    ],
    "env" : {
        "currentCoinbase" : "2adc25665018aa1fe0e6bc666dac8fc2697ff9ba",
        "currentDifficulty" : "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff",
        "currentGasLimit" : "0xe8d4a51000",
        "currentNumber" : "0x012c",
        "currentTimestamp" : "0x02",
        "previousHash" : "5e20a0453cecd065ea59c37ac63e079ee08998b6045136a8ce6635c7912ec0b6"
    },
    "exec" : {
        "address" : "0f572e5295c57f15886f9b263e2f6d2d6c7b5ec6",
        "caller" : "cd1722f3947def4cf144679da39c4c32bdc35681",
        "code" : "0x6000740100000000000000000000000000000000000000007fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff73ffffffffffffffffffffffffffffffffffffffff3c",
        "data" : "0x",
        "gas" : "0xe8d4a51000",
        "gasPrice" : "0x01",
        "origin" : "cd1722f3947def4cf144679da39c4c32bdc35681",
        "value" : "0xe8d4a51000"
    },
    "gas" : "0xe8d4a50fe0",
    "logs" : [
    ],
    "out" : "0x",
    "post" : {
        "0f572e5295c57f15886f9b263e2f6d2d6c7b5ec6" : {
            "balance" : "0x0de0b6b3a7640000",
            "code" : "0x6000740100000000000000000000000000000000000000007fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff73ffffffffffffffffffffffffffffffffffffffff3c",
            "nonce" : "0x00",
            "storage" : {
            }
        },
        "ffffffffffffffffffffffffffffffffffffffff" : {
            "balance" : "0x00",
            "code" : "0x",
            "nonce" : "0x00",
            "storage" : {
            }
        }
    },
    "pre" : {
        "0f572e5295c57f15886f9b263e2f6d2d6c7b5ec6" : {
            "balance" : "0x0de0b6b3a7640000",
            "code" : "0x6000740100000000000000000000000000000000000000007fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff73ffffffffffffffffffffffffffffffffffffffff3c",
            "nonce" : "0x00",
            "storage" : {
            }
        }
    }
},
@pirapira
Copy link
Member

Which file is this from?

@feliam
Copy link
Contributor Author

feliam commented Jul 26, 2017

$ grep 12a1f52c36a8dae436b0d2d354614b5c30deb77a7d104cbf22f6b8ef8c42429c . -Rn
./vmInputLimitsLight.json:27344: "12a1f52c36a8dae436b0d2d354614b5c30deb77a7d104cbf22f6b8ef8c42429c" : {
./vmInputLimits.json:104523: "12a1f52c36a8dae436b0d2d354614b5c30deb77a7d104cbf22f6b8ef8c42429c" : {

@pirapira
Copy link
Member

I think testeth should detect this ethereum/aleth#4281. And then we will need to fix the violating tests including this one.

@rtkaczyk
Copy link

rtkaczyk commented Aug 1, 2017

EXTCODECOPY may create new accounts in these tests. More details here.

@winsvega
Copy link
Collaborator

added some tests around it #329
the related issue is probably about VMTests only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants