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

Pcre threadsafe 4720 v2 #6427

Closed
wants to merge 3 commits into from

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/4720

Describe changes:

  • Creates a pcre2_match_data for each thread to be used in of DetectPcrePayloadMatch
  • Creates a pcre2_match_data for each call to pcerexform Transform (as we do not have access to DetectEngineThreadCtx in transforms)

Replaces #6425 with adding

  • commit using thread storage instead of reallocating locally for each call of the function DetectPcrePayloadMatch
  • commit about pcrexform

So that DetectPcrePayloadMatch is thread safe
and does not rewrite a shared parse_regex.match structure
@catenacyber
Copy link
Contributor Author

Replaced by #6429

jasonish added a commit to jasonish/suricata that referenced this pull request Oct 27, 2023
Also cleanup the wording a little.

Task: OISF#6427
jasonish added a commit to jasonish/suricata that referenced this pull request Oct 27, 2023
These auto modes were remove many years ago. Also cleanup the wording
a little.

Task: OISF#6427
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Oct 30, 2023
These auto modes were remove many years ago. Also cleanup the wording
a little.

Task: OISF#6427
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant