feat(heuristics): add two analyzers to detect dependency confusion and distinguish from stub packages #1117
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
oracle-contributor-agreement
bot
added
the
OCA Verified
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
src/macaron/malware_analyzer/pypi_heuristics/metadata/unknown_organization.py
Fixed
Show fixed
Hide fixed
AmineRaouane
force-pushed
the
dependency-confusion
branch
from
2585e75 to
ab17698
Compare
AmineRaouane
changed the title
…and distinguish from stub packages Signed-off-by: Amine <amine.raouane@enim.ac.ma>
AmineRaouane
force-pushed
the
dependency-confusion
branch
from
ab17698 to
ae021e1
Compare
AmineRaouane
changed the title
AmineRaouane
force-pushed
the
dependency-confusion
branch
from
d555c54 to
7b6f07c
Compare
art1f1c3R
reviewed
Aug 8, 2025
| - **Rule**: Return `HeuristicResult.FAIL` if no descriptive word is found in the package description or summary ; otherwise, return `HeuristicResult.PASS`. | ||
| - **Dependency**: None. | ||
|
|
||
| 13. **Unknown Organization** |
There was a problem hiding this comment.
Is this heuristic going to be included?
There was a problem hiding this comment.
no no a mistake , I forgot to update the README
art1f1c3R
reviewed
Aug 8, 2025
| @@ -411,6 +415,12 @@ def run_check(self, ctx: AnalyzeContext) -> CheckResultData: | |||
| {Confidence.HIGH.value}::trigger(malware_high_confidence_4) :- | |||
| quickUndetailed, forceSetup, failed({Heuristics.TYPOSQUATTING_PRESENCE.value}). | |||
|
|
|||
| % Package released with dependency confusion . | |||
| {Confidence.HIGH.value}::trigger(malware_high_confidence_5) :- | |||
There was a problem hiding this comment.
I'm worried this rule may generate false positives. This is a package with more than 50 files, an anomalous version, and does not include a stub package keyword. Are there any other heuristics or properties we can add on to this rule to be more restrictive?
There was a problem hiding this comment.
maybe forceSetup can be added
Signed-off-by: Amine <amine.raouane@enim.ac.ma>
AmineRaouane
force-pushed
the
dependency-confusion
branch
from
7b6f07c to
157c1c2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds three new heuristic analyzers designed to detect potential dependency confusion attacks and differentiate them from harmless stub or placeholder packages.
Description of changes
heuristics.py.Related issues
None
Checklist
verifiedlabel should appear next to all of your commits on GitHub.