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

processContentOffset Cleanup #1757

Conversation

p0pr0ck5
Copy link
Contributor

@p0pr0ck5 p0pr0ck5 commented Apr 30, 2018

Remove an unused variable, and fix a memory leak.

@p0pr0ck5 p0pr0ck5 changed the title V3/processcontentoffset cleanup processContentoffset cleanup May 1, 2018
@p0pr0ck5 p0pr0ck5 changed the title processContentoffset cleanup processContentOffset Cleanup May 1, 2018
zimmerle pushed a commit that referenced this pull request May 3, 2018
@zimmerle zimmerle self-requested a review May 3, 2018 18:12
@zimmerle zimmerle self-assigned this May 3, 2018
@zimmerle zimmerle added the 3.x Related to ModSecurity version 3.x label May 3, 2018
@zimmerle
Copy link
Contributor

zimmerle commented May 3, 2018

Merged! thanks

@zimmerle zimmerle closed this May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Related to ModSecurity version 3.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants