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

Preallocate memory when SecStreamInBodyInspection is on #1538

Closed
wants to merge 6 commits into from
Closed

Preallocate memory when SecStreamInBodyInspection is on #1538

wants to merge 6 commits into from

Commits on Aug 19, 2017

  1. Preallocate memory when SecStreamInBodyInspection is on. 20x speed im… 

    …provement for 10mb upload. Also simplified modsecurity_request_body_to_stream.
    @allanrbo
    allanrbo committed Aug 19, 2017
    Configuration menu
    Copy the full SHA
    86adee1 View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2017

  1. Renamed local var and initialized local vars. Undid accidental move.

    @allanrbo
    allanrbo committed Aug 22, 2017
    Configuration menu
    Copy the full SHA
    940e2b6 View commit details
    Browse the repository at this point in the history

  2. Ensure memory preallocation for streaming is bounded by SecRequestBod… 

    …yLimit
    @allanrbo
    allanrbo committed Aug 22, 2017
    Configuration menu
    Copy the full SHA
    16c2fcc View commit details
    Browse the repository at this point in the history

  3. Remove the unneeded null termination for the stream_input_data

    @allanrbo
    allanrbo committed Aug 22, 2017
    Configuration menu
    Copy the full SHA
    ba8559a View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2017

  1. Check return value of modsecurity_request_body_store

    @allanrbo
    allanrbo committed Aug 23, 2017
    Configuration menu
    Copy the full SHA
    466bb73 View commit details
    Browse the repository at this point in the history

Commits on Aug 30, 2017

  1. Avoid use of min-macro, as it is not available in all envs

    @allanrbo
    allanrbo committed Aug 30, 2017
    Configuration menu
    Copy the full SHA
    1a865bf View commit details
    Browse the repository at this point in the history