Skip to content

libclamav: Remove confusing heuristic - #1573

Merged
val-ms merged 1 commit into
Cisco-Talos:mainfrom
jhumlick:CLAM-2834-dicsizelimit
Sep 26, 2025
Merged

val-ms merged 1 commit into
Cisco-Talos:mainfrom
jhumlick:CLAM-2834-dicsizelimit

Conversation

@jhumlick

Copy link
Copy Markdown
Contributor

This commit removes the XZ.DicSizeLimit heuristic. The heuristic was triggered whenever CLI_MAX_ALLOCATION was reached or an OOM event happened during one of the allocation wrappers. Due to the heuristic not being indicative of the actual event(s) that lead to its triggering, and that we have not had any valid hits for this heuristic, we are opting to remove it.

CLAM-2834

This commit removes the XZ.DicSizeLimit heuristic. The heuristic was
triggered whenever CLI_MAX_ALLOCATION was reached or an OOM event
happened during one of the allocation wrappers. Due to the heuristic not
being indicative of the actual event(s) that lead to its triggering, and
that we have not had any valid hits for this heuristic, we are opting to
remove it.

CLAM-2834
@jhumlick
jhumlick requested a review from val-ms September 24, 2025 23:07
@jhumlick jhumlick self-assigned this Sep 24, 2025
@val-ms
val-ms merged commit 9691aa7 into Cisco-Talos:main Sep 26, 2025
21 of 24 checks passed
jhumlick added a commit to jhumlick/clamav that referenced this pull request Nov 4, 2025
This commit removes the XZ.DicSizeLimit heuristic. The heuristic was
triggered whenever CLI_MAX_ALLOCATION was reached or an OOM event
happened during one of the allocation wrappers. Due to the heuristic not
being indicative of the actual event(s) that lead to its triggering, and
that we have not had any valid hits for this heuristic, we are opting to
remove it.

CLAM-2834
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants