Page MenuHomePhabricator

Parse UAHF start time as integer constant
ClosedPublic

Authored by freetrader on Jul 11 2017, 10:51.

Details

Summary

There is no need to eval the start time -
just parse it as an integer.

Test Plan

../qa/pull-tester/rpc-tests.py

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptJul 11 2017, 10:51
deadalnix requested changes to this revision.Jul 11 2017, 12:05
deadalnix added inline comments.
qa/rpc-tests/test_framework/cdefs.py
31 ↗(On Diff #790)

This one as well.

This revision now requires changes to proceed.Jul 11 2017, 12:05
qa/rpc-tests/test_framework/cdefs.py
31 ↗(On Diff #790)

To do this one requires changing the C++ code to not use an expression, but a straight numeric constant.

@deadalnix : please confirm whether you really want the consensus.h code changed to turn DEFAULT_MAX_BLOCK_SIZE into a simple number instead of 8 * ONE_MEGABYTE .

freetrader edited edge metadata.

Handing back to review for an answer to the question I asked.

I accept as this is step forward, but I'd like to get rid of eval in there. There is no input validation and there could be any random crap that's evaled.

This revision is now accepted and ready to land.Jul 12 2017, 17:13
This revision was automatically updated to reflect the committed changes.