Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the arguments order of write_sample #17

Merged
merged 1 commit into from Dec 25, 2019
Merged
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Fix the argument order

This was broken by 560c937. This is what happens
when you write your code in a virtual-env,
and port it to your development copy without testing it.
  • Loading branch information
jvoisin committed Dec 24, 2019
commit f41dd5ab0fa82436d23d5ad12f3e4b9dabfcf7e1
@@ -67,7 +67,7 @@ def log_stats(self, log_type):
self._total_executions, log_type, self._total_coverage, self._corpus.length, execs_per_second, rss))
return rss

def write_sample(self, prefix='crash-', buf):
def write_sample(self, buf, prefix='crash-'):
m = hashlib.sha256()
m.update(buf)
if self._exact_artifact_path:
@@ -94,7 +94,7 @@ def start(self):
self._p.kill()
logging.info("=================================================================")
logging.info("timeout reached. testcase took: {}".format(self._timeout))
self.write_sample(prefix='timeout-', buf)
self.write_sample(buf, prefix='timeout-')
break

total_coverage = parent_conn.recv()
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.