Merge pull request #24 from trailofbits/patch/6/ensureconsistency

Fixes #22
2019-06-13 01:15:18 -03:00 · 2019-06-13 01:15:18 -03:00 · 9fcf3fb396
parent 1ab9506883 a8a63d5be9
commit 9fcf3fb396
1 changed files with 2 additions and 1 deletions
--- a/contracts/status/sticker-market/StickerMarket.sol
+++ b/contracts/status/sticker-market/StickerMarket.sol
@ -66,7 +66,8 @@ contract StickerMarket is Controlled, NFTokenEnumerable, ApproveAndCallFallBack
    }

    modifier packOwner(uint256 _packId) {
-        require(msg.sender == controller || packs[_packId].owner == msg.sender);
+        address owner = packs[_packId].owner;
+        require((msg.sender == owner) || (owner != address(0) && msg.sender == controller));
        _;
    }