-
Notifications
You must be signed in to change notification settings - Fork 297
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did not catch the same issue as in referenced gist. The fuzzer caught SEGV.
shared_model::interface::Transaction>> | ||
tx_validator = | ||
std::make_unique<shared_model::validation:: | ||
DefaultOptionalSignedTransactionValidator>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem with this approach is that the possible changes in application.cpp
will not be reflected in this file. Please think of a solution where components in application.cpp
are referenced here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After discussion with @igor-egorov we decided to leave TODO and fix this issue later since it affect other tasks too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix andrei's issue
Signed-off-by: Konstantin Munichev <toobwn@gmail.com>
fa3b6ef
to
b79030c
Compare
Signed-off-by: Konstantin Munichev <toobwn@gmail.com>
b79030c
to
3f0b667
Compare
Description of the Change
Fuzzing for MST endpoint (SendState).
Benefits
One more endpoint covered.
Possible Drawbacks
CI is needed.
Also it looks like a false positive warning from Address Sanitizer (at least I couldn't find a problem - see https://gist.github.com/luckychess/b65bd40e958887273e86a0a535cc896d).
Usage Examples or Tests