Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

More fuzzing fixes #1984

Merged
merged 6 commits into from
Dec 25, 2018
Merged

More fuzzing fixes #1984

merged 6 commits into from
Dec 25, 2018

Conversation

luckychess
Copy link
Contributor

Description of the Change

Pull request fixes send_batches_fuzz and mst_fuzz.

Benefits

Working fuzzing, CI should pass now.

Possible Drawbacks

None.

Signed-off-by: Konstantin Munichev <toobwn@gmail.com>
Signed-off-by: Konstantin Munichev <toobwn@gmail.com>
Copy link
Contributor

@kamilsa kamilsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please set log level, so that warnings are not seen

@neewy neewy added this to the rc2 milestone Dec 21, 2018
Signed-off-by: Konstantin Munichev <toobwn@gmail.com>
@l4l l4l added tests pr aimed at the code coverage increase or test refactorings security labels Dec 21, 2018
Copy link
Contributor

@l4l l4l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an issue that can be found by that fuzzer easily. Please add a task for fixing it

test/fuzzing/mst_fuzz.cpp Outdated Show resolved Hide resolved
@luckychess luckychess merged commit 311f897 into dev Dec 25, 2018
@luckychess luckychess deleted the fix/fuzzing_new branch December 25, 2018 10:29
@luckychess
Copy link
Contributor Author

@l4l What exact issue you mean?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
security tests pr aimed at the code coverage increase or test refactorings
Development

Successfully merging this pull request may close these issues.

None yet

4 participants