check for contract code ending in an empty push pre fork 10

[hexoscott]

No description provided.

[afa7789]

shouldn't we be using the push code reference here instead of hardcoded number ?

something like :
"github.com/ledgerwatch/erigon/core/vm" with vm.PUSH1 not sure if this is it tough.

or is it cause it can be any of these ?

[hexoscott]

Using the variable over a magic number would be nice 👍 I guess it could be any of these push opcodes but the original issue was specifically for a PUSH1 without anything to actually push to the stack after it, like a trailing push. Probably a good idea to check with Ignasi at Polygon, we might need something more sophisticated to handle the other push opcodes.