Skip to content

Commit ac4d38c

Browse files
author
MarcoFalke
committed
Merge #16236: fuzz: Log output even if fuzzer failed
fa410f6 test: Suppress false positive leak in secure_allocator<RNGState> (MarcoFalke) fa35c42 test: Log output even if fuzzer failed (MarcoFalke) Pull request description: Also suppress a false positive detected leak ACKs for commit fa410f: practicalswift: utACK fa410f6 Tree-SHA512: 224a72ae0dd9bbe7debda17cd626c01cfbd0e45d7df47a2b591ce8ea386951ad94f4c0677dd268079a4caac382c5acac03199146015a95c308a633e9e4f84c09
2 parents 0b68fca + fa410f6 commit ac4d38c

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

test/fuzz/test_runner.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,10 @@ def run_once(*, corpus, test_list, build_dir, export_coverage):
106106
os.path.join(corpus, t),
107107
]
108108
logging.debug('Run {} with args {}'.format(t, args))
109-
output = subprocess.run(args, check=True, stderr=subprocess.PIPE, universal_newlines=True).stderr
109+
result = subprocess.run(args, stderr=subprocess.PIPE, universal_newlines=True)
110+
output = result.stderr
110111
logging.debug('Output: {}'.format(output))
112+
result.check_returncode()
111113
if not export_coverage:
112114
continue
113115
for l in output.splitlines():

test/sanitizer_suppressions/lsan

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,6 @@ leak:libqminimal
44
leak:libQt5Core
55
leak:libQt5Gui
66
leak:libQt5Widgets
7+
8+
# false-positive due to use of secure_allocator<>
9+
leak:GetRNGState

0 commit comments

Comments
 (0)