CORIONplatform / solidity

GNU General Public License v3.0
12 stars 9 forks source link

publisher: transferEvent method implementation is broken #96

Closed gundas closed 7 years ago

gundas commented 7 years ago

There are 4 typos in the publisher contract's transferEvent method:

1. https://github.com/CORIONplatform/solidity/blob/master/publisher.sol#L22 should be: announcementID = opponents[from][a];

2. https://github.com/CORIONplatform/solidity/blob/master/publisher.sol#L24 should be: announcements[announcementID].oppositionWeight = safeSub(announcements[announcementID].oppositionWeight, value);

3. https://github.com/CORIONplatform/solidity/blob/master/publisher.sol#L28 should be: announcementID = opponents[to][a];

4. https://github.com/CORIONplatform/solidity/blob/master/publisher.sol#L30 should be: announcements[announcementID].oppositionWeight = safeAdd(announcements[announcementID].oppositionWeight, value);

P.S. It was tempting to create 4 bugs for the sake of bounty :)