Skip to content

Conversation

@owen-mc
Copy link
Contributor

@owen-mcowen-mc commented Nov 18, 2025

Currently Go and Javascript have this behaviour without any problems, but we have to check that it doesn't lead to FPs (or at least that they're balanced out by enough TPs) or any performance problems.

@owen-mcowen-mcforce-pushed the java/propagate-taint-through-field-reads branch from 6f8a6c1 to 711d8daCompareNovember 28, 2025 09:59
@owen-mc
Copy link
ContributorAuthor

owen-mc commented Nov 28, 2025

DCA shows an increase in analysis time of 13%-320%, plus one workflow failure:

[255/255 eval 230m25s] Query failed: codeql/java-queries/Security/CWE/CWE-532/SensitiveInfoLog.ql (Query evaluation ran out of Java heap (Java heap maximum: 3441 MiB). The evaluator previously allocated beyond the heap limit. (eventual cause: OutOfMemoryError "Java heap space")). CodeQL is out of memory. Try running CodeQL on a larger runner (hosted or self-hosted). If you continue to encounter this issue, contact GitHub Support. 

This might be fixable, with some investigation.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@owen-mc