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.

Sign up for GitHub

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

Jump to bottom

RFC: Add fuzzing tests #402

Closed
wants to merge 3 commits into from
Closed

RFC: Add fuzzing tests #402

wants to merge 3 commits into from

Commits on Jul 23, 2018

  1. Change modbus_get_header_length to be unsigned 

    Change          int modbus_get_header_length(modbus_t *ctx)
to     unsigned int modbus_get_header_length(modbus_t *ctx)

The result of modbus_get_header_length can never be negative and seems
to be on signed due to a internal use of -1 in some places outside the
public API

This commit also follows the minimum number of signed -> unsigned
changes without control flow changes.
    @TheBiggerGuy
    TheBiggerGuy committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    fa203f7 View commit details
    Browse the repository at this point in the history

  2. First attempt to add fuzzing 

    This is a first attempt at adding a fuzzing harness to the test suite.
It uses the "American Fuzzy Lop" (AFL) fuzzer and only requires a small
harness to test a TCP ModBus server.
    @TheBiggerGuy
    TheBiggerGuy committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    8e8b18d View commit details
    Browse the repository at this point in the history

  3. Improve fuzzing_harness 

    This enables the faster "clang-fast" compiler as well as better resting
server state between runs.
    @TheBiggerGuy
    TheBiggerGuy committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    d23ecbe View commit details
    Browse the repository at this point in the history