Skip to content

Conversation

@akosthekiss
Copy link
Member

  • Allow command line tools to build together with libfuzzer driver.
    Compile everything with -fsanitize=fuzzer-no-link to prevent
    linking in libfuzzers's main symbol in all executables (causing
    duplicate symbol errors in command line tools), and add
    -fsanitize=fuzzer to the libfuzzer driver only.

  • Make ASan optional when building with libfuzzer to allow the user
    to choose freely from available sanitizers (e.g., UBSan, MSan,
    HWASan).

  • Stabilize libfuzzer by resetting PRNG seed at every invocation.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss [email protected]

- Allow command line tools to build together with libfuzzer driver.
  Compile everything with `-fsanitize=fuzzer-no-link` to prevent
  linking in libfuzzers's `main` symbol in all executables (causing
  duplicate symbol errors in command line tools), and add
  `-fsanitize=fuzzer` to the libfuzzer driver only.

- Make ASan optional when building with libfuzzer to allow the user
  to choose freely from available sanitizers (e.g., UBSan, MSan,
  HWASan).

- Stabilize libfuzzer by resetting PRNG seed at every invocation.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss [email protected]
@akosthekiss akosthekiss added the test Related to testing label Jun 18, 2019
Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rerobika rerobika left a comment

Choose a reason for hiding this comment

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

LGTM

@rerobika rerobika merged commit 8ee8bc2 into jerryscript-project:master Jun 19, 2019
@akosthekiss akosthekiss deleted the libfuzz-cmdline-all branch June 19, 2019 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants