Skip to content

[ci] try to fix the fuzz coverage failure#13175

Merged
mattklein123 merged 4 commits intoenvoyproxy:masterfrom
asraa:fix-fuzz-coverage-2
Sep 18, 2020
Merged

[ci] try to fix the fuzz coverage failure#13175
mattklein123 merged 4 commits intoenvoyproxy:masterfrom
asraa:fix-fuzz-coverage-2

Conversation

@asraa
Copy link
Copy Markdown
Contributor

@asraa asraa commented Sep 18, 2020

Signed-off-by: Asra Ali asraa@google.com

Attempt to fix #13171
I made used the actual regression testing process (using a list of files as args) instead of running with -runs=0 which seems to be doing some (preliminary?) fuzzing process. This seems to avoid catching bugs while capturing coverage.

Even then, sometimes there are flakes (memory corruption), particularly in network_readfilter_fuzz_test (see below). Added a workaround to always pass.

2020-09-18T20:30:05.1746225Z UndefinedBehaviorSanitizer:DEADLYSIGNAL
2020-09-18T20:30:05.1747002Z ==27421==ERROR: UndefinedBehaviorSanitizer: SEGV on unknown address 0x000000000000 (pc 0x000000000000 bp 0x7fffbf9fe9e0 sp 0x7fffbf9fe7f8 T27421)
2020-09-18T20:30:05.1747878Z ==27421==Hint: pc points to the zero page.
2020-09-18T20:30:05.1748440Z ==27421==The signal is caused by a READ memory access.
2020-09-18T20:30:05.1748996Z ==27421==Hint: address points to the zero page.
2020-09-18T20:30:05.1749664Z     #0 0x0  (<unknown module>)
2020-09-18T20:30:05.1751543Z     #1 0x41ff2a  (/build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/k8-fastbuild/bin/test/extensions/filters/network/common/fuzz/network_readfilter_fuzz_test+0x41ff2a)
2020-09-18T20:30:05.1753115Z     #2 0x7fb8c3c7489f  (/lib/x86_64-linux-gnu/libpthread.so.0+0x1289f)
2020-09-18T20:30:05.1753581Z 
2020-09-18T20:30:05.1754055Z UndefinedBehaviorSanitizer can not provide additional info.
2020-09-18T20:30:05.1754706Z SUMMARY: UndefinedBehaviorSanitizer: SEGV (<unknown module>) 
2020-09-18T20:30:05.1755253Z ==27421==ABORTING

Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
@asraa
Copy link
Copy Markdown
Contributor Author

asraa commented Sep 18, 2020

@envoyproxy/maintainers

@mattklein123 mattklein123 merged commit ba66a31 into envoyproxy:master Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ci] fuzz coverage will test against crashing entries after fuzzing

2 participants