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

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

Closed
wants to merge 4 commits into from

Commits on Jun 21, 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 committed Jun 21, 2023
    Configuration menu
    Copy the full SHA
    aa2e723 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 committed Jun 21, 2023
    Configuration menu
    Copy the full SHA
    633ef85 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 committed Jun 21, 2023
    Configuration menu
    Copy the full SHA
    42966b3 View commit details
    Browse the repository at this point in the history
  4. detect/cleanup: Minor cleanups

    Move declarations closer to usage.
    jlucovsky committed Jun 21, 2023
    Configuration menu
    Copy the full SHA
    4a700f4 View commit details
    Browse the repository at this point in the history