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

detect/multi-tenant: Eliminate crashes processing tenant config #9060

Closed
wants to merge 3 commits into from
Closed

detect/multi-tenant: Eliminate crashes processing tenant config #9060

wants to merge 3 commits into from

Commits on Jun 22, 2023

  1. detect/multi-tenant: Make tenant_id 32 bits everywhere 

    Issue: 6047

This commit ensures that the tenant id is contained in a unsigned 32 bit
container.
    @jlucovsky
    jlucovsky committed Jun 22, 2023
    Configuration menu
    Copy the full SHA
    0ccdaa6 View commit details
    Browse the repository at this point in the history

  2. detect/config: Serialize config file processing 

    Issue: 6047

This commit serializes processing of individual .config file processing.
This is of little consequence when multi-tenancy is not in use.

When multiple tenants have been configured, serialization will ensure
parallel loading of configuration/rules doesn't concurrently use shared
pcre2 constructs.
    @jlucovsky
    jlucovsky committed Jun 22, 2023
    Configuration menu
    Copy the full SHA
    0721a5e View commit details
    Browse the repository at this point in the history

  3. detect/alloc: Use SCCalloc instead of SCMalloc 

    This commit changes memory allocations to SCCalloc for clarity.
    @jlucovsky
    jlucovsky committed Jun 22, 2023
    Configuration menu
    Copy the full SHA
    b8c5728 View commit details
    Browse the repository at this point in the history