No range checks in NativeCurrency

[Sword-Smith]

There are to loops in NativeCurrency where there are no range checks on the size indicators of dynamically-sized structs. These size indicators are, AFAICT, used to find the next element in a list, and should thus be range-checked.

[elielnfinic]

I am willing to work on this.

[aszepieniec]

Hi @elielnfinic! Sorry to inform that this issue has already been fixed by 9368dd7. The reason why this issue is still open is because that commit lives on a branch that has not been merged into master yet, but soon will be.

If you are eager to contribute, here are some high-priority open issues:

• Node Hangs #282 (and possibly related: Problem with bootstrapping? #286) involving application programming, concurrency, write/read locks
• Bootstrapper Mode Refuses Incoming Connections #287 involving application programming, concurrency
• Node crashes in peer_loop #285 involving application programming
• Trigger Untriggered Error IDs in NativeCurrency with Dedicated Negative Proptests #288 involving consensus-critical logic, tasm code, proptests
• Fix NeptuneCoins #283 Note that branch asz/negative-fees already changed the inner type but we still need and want a very strong test suite for NeptuneCoins

[elielnfinic]

Alright, I will pick some tasks. Btw, it would be good to also share these tasks on the Telegram group(maybe??).