muaddib-scanner 2.11.113 → 2.11.115
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/audit-data/adjudication-2026-06-14.json +56 -0
- package/audit-data/fpr-baseline-2026-06-14.json +2648 -0
- package/package.json +1 -1
- package/{self-scan-v2.11.113.json → self-scan-v2.11.115.json} +18 -13
- package/src/intent-graph.js +34 -192
- package/src/pipeline/executor.js +5 -1
- package/src/pipeline/processor.js +15 -7
- package/src/scanner/ast-detectors/handle-post-walk.js +9 -2
- package/src/scanner/module-graph/annotate-sinks.js +8 -5
- package/src/scanner/module-graph/constants.js +10 -1
- package/src/scanner/module-graph/detect-cross-file.js +56 -4
- package/src/scanner/module-graph/index.js +2 -2
- package/src/scanner/module-graph/parse-utils.js +13 -1
- package/src/scoring.js +41 -0
- package/src/sdk-destination.js +328 -0
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"target": "node_modules",
|
|
3
|
-
"timestamp": "2026-06-
|
|
3
|
+
"timestamp": "2026-06-14T16:22:45.950Z",
|
|
4
4
|
"threats": [
|
|
5
5
|
{
|
|
6
6
|
"type": "string_mutation_obfuscation",
|
|
@@ -244,7 +244,7 @@
|
|
|
244
244
|
},
|
|
245
245
|
{
|
|
246
246
|
"type": "credential_regex_harvest",
|
|
247
|
-
"severity": "
|
|
247
|
+
"severity": "LOW",
|
|
248
248
|
"message": "Credential regex patterns (token/password/secret/Bearer) + network call in same file — stream data credential harvesting.",
|
|
249
249
|
"file": "web-tree-sitter/web-tree-sitter.cjs",
|
|
250
250
|
"count": 1,
|
|
@@ -252,6 +252,11 @@
|
|
|
252
252
|
{
|
|
253
253
|
"rule": "count_threshold_floor",
|
|
254
254
|
"note": "retained one instance at original severity"
|
|
255
|
+
},
|
|
256
|
+
{
|
|
257
|
+
"rule": "sink_coupling",
|
|
258
|
+
"from": "HIGH",
|
|
259
|
+
"to": "LOW"
|
|
255
260
|
}
|
|
256
261
|
],
|
|
257
262
|
"originalSeverity": "HIGH",
|
|
@@ -266,7 +271,7 @@
|
|
|
266
271
|
],
|
|
267
272
|
"mitre": "T1552",
|
|
268
273
|
"playbook": "Code contient des regex de detection de credentials (Bearer, password, token, API key) combine avec un appel reseau. Technique de harvesting: scanne les donnees en transit (streams HTTP, fichiers) pour extraire des secrets et les exfiltrer. Supprimer le package. Auditer le trafic reseau sortant.",
|
|
269
|
-
"points":
|
|
274
|
+
"points": 1
|
|
270
275
|
},
|
|
271
276
|
{
|
|
272
277
|
"type": "proxy_data_intercept",
|
|
@@ -1823,18 +1828,18 @@
|
|
|
1823
1828
|
"summary": {
|
|
1824
1829
|
"total": 80,
|
|
1825
1830
|
"critical": 7,
|
|
1826
|
-
"high":
|
|
1831
|
+
"high": 10,
|
|
1827
1832
|
"medium": 32,
|
|
1828
|
-
"low":
|
|
1833
|
+
"low": 31,
|
|
1829
1834
|
"riskScore": 35,
|
|
1830
1835
|
"riskLevel": "MEDIUM",
|
|
1831
1836
|
"globalRiskScore": 100,
|
|
1832
|
-
"maxFileScore":
|
|
1837
|
+
"maxFileScore": 42,
|
|
1833
1838
|
"packageScore": 1,
|
|
1834
1839
|
"mostSuspiciousFile": "web-tree-sitter/web-tree-sitter.cjs",
|
|
1835
1840
|
"fileScores": {
|
|
1836
1841
|
"esquery/parser.js": 5,
|
|
1837
|
-
"web-tree-sitter/web-tree-sitter.cjs":
|
|
1842
|
+
"web-tree-sitter/web-tree-sitter.cjs": 42,
|
|
1838
1843
|
"web-tree-sitter/web-tree-sitter.js": 42,
|
|
1839
1844
|
"ajv/lib/ajv.js": 25,
|
|
1840
1845
|
"ajv/scripts/bundle.js": 10,
|
|
@@ -1927,12 +1932,6 @@
|
|
|
1927
1932
|
"points": 10,
|
|
1928
1933
|
"reason": "Binary file reference (.png/.jpg/.wasm/etc.) + eval() in same file — possible steganographic payload execution."
|
|
1929
1934
|
},
|
|
1930
|
-
{
|
|
1931
|
-
"rule": "MUADDIB-AST-041",
|
|
1932
|
-
"type": "credential_regex_harvest",
|
|
1933
|
-
"points": 10,
|
|
1934
|
-
"reason": "Credential regex patterns (token/password/secret/Bearer) + network call in same file — stream data credential harvesting."
|
|
1935
|
-
},
|
|
1936
1935
|
{
|
|
1937
1936
|
"rule": "MUADDIB-AST-020",
|
|
1938
1937
|
"type": "staged_binary_payload",
|
|
@@ -2185,6 +2184,12 @@
|
|
|
2185
2184
|
"points": 1,
|
|
2186
2185
|
"reason": "Dangerous call \"eval\" with dynamic expression detected."
|
|
2187
2186
|
},
|
|
2187
|
+
{
|
|
2188
|
+
"rule": "MUADDIB-AST-041",
|
|
2189
|
+
"type": "credential_regex_harvest",
|
|
2190
|
+
"points": 1,
|
|
2191
|
+
"reason": "Credential regex patterns (token/password/secret/Bearer) + network call in same file — stream data credential harvesting."
|
|
2192
|
+
},
|
|
2188
2193
|
{
|
|
2189
2194
|
"rule": "MUADDIB-AST-043",
|
|
2190
2195
|
"type": "proxy_data_intercept",
|
package/src/intent-graph.js
CHANGED
|
@@ -36,180 +36,17 @@ const SOURCE_TYPES = {
|
|
|
36
36
|
// Sensitive env var patterns — env_access referencing these is credential theft, not config
|
|
37
37
|
const SENSITIVE_ENV_PATTERNS = /TOKEN|KEY|SECRET|PASSWORD|CREDENTIAL|API_KEY|AUTH/i;
|
|
38
38
|
|
|
39
|
-
//
|
|
40
|
-
//
|
|
41
|
-
//
|
|
42
|
-
//
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
{ envPattern: /^SALESFORCE_/i, domains: ['salesforce.com', 'force.com'] },
|
|
51
|
-
{ envPattern: /^SUPABASE_/i, domains: ['supabase.co', 'supabase.com'] },
|
|
52
|
-
{ envPattern: /^MAILGUN_/i, domains: ['mailgun.net', 'mailgun.com'] },
|
|
53
|
-
{ envPattern: /^STRIPE_/i, domains: ['stripe.com'] },
|
|
54
|
-
{ envPattern: /^TWILIO_/i, domains: ['twilio.com'] },
|
|
55
|
-
{ envPattern: /^SENDGRID_/i, domains: ['sendgrid.com', 'sendgrid.net'] },
|
|
56
|
-
{ envPattern: /^DATADOG_/i, domains: ['datadoghq.com'] },
|
|
57
|
-
{ envPattern: /^SENTRY_/i, domains: ['sentry.io'] },
|
|
58
|
-
{ envPattern: /^SLACK_/i, domains: ['slack.com'] },
|
|
59
|
-
{ envPattern: /^GITHUB_/i, domains: ['github.com', 'githubusercontent.com'] },
|
|
60
|
-
{ envPattern: /^GITLAB_/i, domains: ['gitlab.com'] },
|
|
61
|
-
{ envPattern: /^CLOUDFLARE_/i, domains: ['cloudflare.com'] },
|
|
62
|
-
{ envPattern: /^OPENAI_/i, domains: ['openai.com'] },
|
|
63
|
-
{ envPattern: /^ANTHROPIC_/i, domains: ['anthropic.com'] },
|
|
64
|
-
{ envPattern: /^MONGODB_|^MONGO_/i, domains: ['mongodb.com', 'mongodb.net'] },
|
|
65
|
-
{ envPattern: /^AUTH0_/i, domains: ['auth0.com'] },
|
|
66
|
-
{ envPattern: /^HUBSPOT_/i, domains: ['hubspot.com', 'hubapi.com'] },
|
|
67
|
-
{ envPattern: /^CONTENTFUL_/i, domains: ['contentful.com'] },
|
|
68
|
-
];
|
|
69
|
-
|
|
70
|
-
// Tokens stripped when extracting brand keyword from env var name
|
|
71
|
-
const ENV_NOISE_TOKENS = new Set([
|
|
72
|
-
'API', 'KEY', 'SECRET', 'TOKEN', 'PASSWORD', 'CREDENTIAL',
|
|
73
|
-
'AUTH', 'ACCESS', 'PRIVATE', 'PUBLIC', 'CLIENT', 'ID', 'URL'
|
|
74
|
-
]);
|
|
75
|
-
|
|
76
|
-
// Suspicious tunneling/proxy domains — never considered legitimate SDK destinations
|
|
77
|
-
const SUSPICIOUS_DOMAIN_PATTERNS = /ngrok|serveo|localtunnel|burpcollaborator|requestbin|pipedream|webhook\.site/i;
|
|
78
|
-
|
|
79
|
-
// URL extraction regex (matches http/https URLs in source code)
|
|
80
|
-
const URL_EXTRACT_RE = /https?:\/\/[a-zA-Z0-9\-._~:/?#[\]@!$&'()*+,;=%]+/g;
|
|
81
|
-
|
|
82
|
-
// Hostname extraction from Node.js request options: hostname: 'domain.com' or host: 'domain.com'
|
|
83
|
-
const HOSTNAME_OPTION_RE = /(?:hostname|host)\s*:\s*['"`]([a-zA-Z0-9\-._]+)['"`]/g;
|
|
84
|
-
|
|
85
|
-
/**
|
|
86
|
-
* Extract env var name from an intent source threat message.
|
|
87
|
-
* Messages look like: "process.env.SALESFORCE_API_KEY", "env var MAILGUN_API_KEY accessed"
|
|
88
|
-
*/
|
|
89
|
-
function extractEnvVarFromMessage(sourceThreats) {
|
|
90
|
-
for (const t of sourceThreats) {
|
|
91
|
-
if (!t.message) continue;
|
|
92
|
-
// Match process.env.VAR_NAME pattern
|
|
93
|
-
const envMatch = t.message.match(/process\.env\.([A-Z_][A-Z0-9_]*)/i);
|
|
94
|
-
if (envMatch) return envMatch[1];
|
|
95
|
-
// Match standalone VAR_NAME patterns (e.g., "SALESFORCE_API_KEY")
|
|
96
|
-
const varMatch = t.message.match(/\b([A-Z][A-Z0-9]*(?:_[A-Z0-9]+)+)\b/);
|
|
97
|
-
if (varMatch) return varMatch[1];
|
|
98
|
-
}
|
|
99
|
-
return null;
|
|
100
|
-
}
|
|
101
|
-
|
|
102
|
-
/**
|
|
103
|
-
* Extract brand keyword from env var name by removing noise tokens.
|
|
104
|
-
* MAILGUN_API_KEY → MAILGUN, SALESFORCE_CLIENT_SECRET → SALESFORCE
|
|
105
|
-
*/
|
|
106
|
-
function extractBrandFromEnvVar(envVarName) {
|
|
107
|
-
const parts = envVarName.toUpperCase().split('_');
|
|
108
|
-
const brandParts = parts.filter(p => !ENV_NOISE_TOKENS.has(p) && p.length > 0);
|
|
109
|
-
return brandParts.length > 0 ? brandParts[0] : null;
|
|
110
|
-
}
|
|
111
|
-
|
|
112
|
-
/**
|
|
113
|
-
* Extract domain from a URL string.
|
|
114
|
-
* Returns the hostname (without port).
|
|
115
|
-
*/
|
|
116
|
-
function extractDomain(url) {
|
|
117
|
-
try {
|
|
118
|
-
const match = url.match(/^https?:\/\/([^/:?#]+)/i);
|
|
119
|
-
return match ? match[1].toLowerCase() : null;
|
|
120
|
-
} catch {
|
|
121
|
-
return null;
|
|
122
|
-
}
|
|
123
|
-
}
|
|
124
|
-
|
|
125
|
-
/**
|
|
126
|
-
* Check if a domain matches any of the expected SDK domains (suffix match).
|
|
127
|
-
* api.mailgun.net matches mailgun.net, sub.api.stripe.com matches stripe.com
|
|
128
|
-
*/
|
|
129
|
-
function domainMatchesSuffix(domain, expectedDomains) {
|
|
130
|
-
for (const expected of expectedDomains) {
|
|
131
|
-
if (domain === expected || domain.endsWith('.' + expected)) return true;
|
|
132
|
-
}
|
|
133
|
-
return false;
|
|
134
|
-
}
|
|
135
|
-
|
|
136
|
-
/**
|
|
137
|
-
* Check if an env var + file content represents a legitimate SDK pattern.
|
|
138
|
-
*
|
|
139
|
-
* Returns true ONLY if:
|
|
140
|
-
* 1. The env var matches a known SDK mapping (allowlist) OR heuristic brand match
|
|
141
|
-
* 2. ALL URLs in the file point to domains matching the expected SDK
|
|
142
|
-
* 3. No suspicious tunneling/proxy domains are present
|
|
143
|
-
*
|
|
144
|
-
* @param {string} envVarName - e.g., "SALESFORCE_API_KEY"
|
|
145
|
-
* @param {string} fileContent - source code of the file
|
|
146
|
-
* @returns {boolean} true if SDK pattern (should skip intent pair)
|
|
147
|
-
*/
|
|
148
|
-
function isSDKPattern(envVarName, fileContent) {
|
|
149
|
-
// Extract domains from full URLs (https://api.stripe.com/v1/charges)
|
|
150
|
-
const urls = fileContent.match(URL_EXTRACT_RE) || [];
|
|
151
|
-
const domains = urls.map(u => extractDomain(u)).filter(Boolean);
|
|
152
|
-
|
|
153
|
-
// Also extract hostnames from Node.js request options (hostname: 'api.stripe.com')
|
|
154
|
-
let hostnameMatch;
|
|
155
|
-
const hostnameRe = new RegExp(HOSTNAME_OPTION_RE.source, 'g');
|
|
156
|
-
while ((hostnameMatch = hostnameRe.exec(fileContent)) !== null) {
|
|
157
|
-
const hostname = hostnameMatch[1].toLowerCase();
|
|
158
|
-
if (hostname && !domains.includes(hostname)) {
|
|
159
|
-
domains.push(hostname);
|
|
160
|
-
}
|
|
161
|
-
}
|
|
162
|
-
|
|
163
|
-
// No URLs found — can't confirm SDK pattern, default to suspicious
|
|
164
|
-
if (domains.length === 0) return false;
|
|
165
|
-
|
|
166
|
-
// Check for suspicious tunneling domains — immediate fail
|
|
167
|
-
for (const domain of domains) {
|
|
168
|
-
if (SUSPICIOUS_DOMAIN_PATTERNS.test(domain)) return false;
|
|
169
|
-
}
|
|
170
|
-
|
|
171
|
-
// Check for raw IP addresses — immediate fail
|
|
172
|
-
for (const domain of domains) {
|
|
173
|
-
if (/^\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}$/.test(domain)) return false;
|
|
174
|
-
}
|
|
175
|
-
|
|
176
|
-
// 1. Try curated allowlist first (strict: ALL domains must match)
|
|
177
|
-
// Curated allowlist is authoritative — no relaxation here to prevent
|
|
178
|
-
// attacker injecting a legitimate domain alongside their C2 domain.
|
|
179
|
-
for (const mapping of SDK_ENV_DOMAIN_MAP) {
|
|
180
|
-
if (mapping.envPattern.test(envVarName)) {
|
|
181
|
-
return domains.every(d => domainMatchesSuffix(d, mapping.domains));
|
|
182
|
-
}
|
|
183
|
-
}
|
|
184
|
-
|
|
185
|
-
// R2: credential-suffixed env vars get relaxed domain matching (at least ONE match).
|
|
186
|
-
// SDKs commonly call their own API + CDN/logging/analytics domains.
|
|
187
|
-
// Safety: suspicious domains and raw IPs are already rejected above.
|
|
188
|
-
// Only applies to the heuristic fallback — curated allowlist stays strict.
|
|
189
|
-
const CREDENTIAL_SUFFIXES = ['_API_KEY', '_SECRET', '_TOKEN', '_SECRET_KEY', '_ACCESS_KEY'];
|
|
190
|
-
const upperName = envVarName.toUpperCase();
|
|
191
|
-
const hasCredentialSuffix = CREDENTIAL_SUFFIXES.some(s => upperName.endsWith(s));
|
|
192
|
-
|
|
193
|
-
// 2. Heuristic fallback: extract brand keyword and check domain labels
|
|
194
|
-
const brand = extractBrandFromEnvVar(envVarName);
|
|
195
|
-
if (!brand || brand.length < 3) return false; // Too short for reliable matching
|
|
196
|
-
|
|
197
|
-
const brandLower = brand.toLowerCase();
|
|
198
|
-
// 2a. Strict check: every domain matches brand (existing behavior)
|
|
199
|
-
// e.g., brand "ACME" matches "api.acme.com" (label "acme") but not "api.acmetech.com"
|
|
200
|
-
if (domains.every(d => {
|
|
201
|
-
const labels = d.split('.');
|
|
202
|
-
return labels.some(label => label === brandLower);
|
|
203
|
-
})) return true;
|
|
204
|
-
|
|
205
|
-
// 2b. R2 relaxed: credential suffix + at least one domain matches brand
|
|
206
|
-
if (hasCredentialSuffix && domains.some(d => {
|
|
207
|
-
const labels = d.split('.');
|
|
208
|
-
return labels.some(label => label === brandLower);
|
|
209
|
-
})) return true;
|
|
210
|
-
|
|
211
|
-
return false;
|
|
212
|
-
}
|
|
39
|
+
// Destination-aware SDK detection — extracted to a shared leaf module
|
|
40
|
+
// (src/sdk-destination.js) so the same logic gates dataflow.js and the cross-file /
|
|
41
|
+
// detached taint detectors, not just intent coherence. Re-exported below for
|
|
42
|
+
// backward compatibility (dataflow.js imports isSDKPattern from this module).
|
|
43
|
+
const {
|
|
44
|
+
isSDKPattern,
|
|
45
|
+
networkDestinationsAllBenign,
|
|
46
|
+
extractEnvVarFromMessage,
|
|
47
|
+
extractBrandFromEnvVar,
|
|
48
|
+
SDK_ENV_DOMAIN_MAP,
|
|
49
|
+
} = require('./sdk-destination.js');
|
|
213
50
|
|
|
214
51
|
|
|
215
52
|
// ============================================
|
|
@@ -384,26 +221,31 @@ function buildIntentPairs(threats, targetPath) {
|
|
|
384
221
|
const pairKey = `${srcType}:${sinkType}:${file}`;
|
|
385
222
|
if (pairSet.has(pairKey)) continue;
|
|
386
223
|
|
|
387
|
-
// Destination-aware
|
|
388
|
-
//
|
|
224
|
+
// Destination-aware check: credential_read → network_external. Two
|
|
225
|
+
// complementary gates, EITHER ⇒ legitimate, skip the pair:
|
|
226
|
+
// (1) isSDKPattern — per-env-var: the env var brand matches its API domain
|
|
227
|
+
// (e.g. STRIPE_API_KEY → stripe.com).
|
|
228
|
+
// (2) networkDestinationsAllBenign — env-var-independent: EVERY network host
|
|
229
|
+
// in the file is a provider/local/reserved destination. Catches multi-
|
|
230
|
+
// provider CLIs (reads GEMINI_API_KEY *and* ANTHROPIC_API_KEY, calls both)
|
|
231
|
+
// and providers absent from the curated env→domain map. Same anti-evasion
|
|
232
|
+
// floor (any suspicious/unknown/public-IP host ⇒ keep firing).
|
|
389
233
|
if (srcType === 'credential_read' && sinkType === 'network_external' && targetPath) {
|
|
390
|
-
|
|
391
|
-
|
|
392
|
-
|
|
393
|
-
|
|
394
|
-
|
|
395
|
-
|
|
396
|
-
|
|
397
|
-
|
|
398
|
-
|
|
399
|
-
|
|
400
|
-
|
|
401
|
-
|
|
402
|
-
continue;
|
|
403
|
-
}
|
|
404
|
-
} catch {
|
|
405
|
-
// File read error — default to suspicious (CRITICAL)
|
|
234
|
+
try {
|
|
235
|
+
let content = fileContentCache.get(file);
|
|
236
|
+
if (content === undefined) {
|
|
237
|
+
const filePath = path.join(targetPath, file);
|
|
238
|
+
content = fs.readFileSync(filePath, 'utf8');
|
|
239
|
+
fileContentCache.set(file, content);
|
|
240
|
+
}
|
|
241
|
+
const envVarName = extractEnvVarFromMessage(sourceThreats);
|
|
242
|
+
if ((envVarName && isSDKPattern(envVarName, content)) || networkDestinationsAllBenign(content)) {
|
|
243
|
+
// First-party/SDK destination — skip this pair
|
|
244
|
+
pairSet.add(pairKey); // Mark as seen to avoid re-checking
|
|
245
|
+
continue;
|
|
406
246
|
}
|
|
247
|
+
} catch {
|
|
248
|
+
// File read error — default to suspicious (CRITICAL)
|
|
407
249
|
}
|
|
408
250
|
}
|
|
409
251
|
|
package/src/pipeline/executor.js
CHANGED
|
@@ -17,7 +17,7 @@ const { scanGitHubActions } = require('../scanner/github-actions.js');
|
|
|
17
17
|
const { scanEntropy } = require('../scanner/entropy.js');
|
|
18
18
|
const { scanAIConfig } = require('../scanner/ai-config.js');
|
|
19
19
|
const { deobfuscate } = require('../scanner/deobfuscate.js');
|
|
20
|
-
const { buildModuleGraph, annotateTaintedExports, detectCrossFileFlows, annotateSinkExports, detectCallbackCrossFileFlows, detectEventEmitterFlows } = require('../scanner/module-graph');
|
|
20
|
+
const { buildModuleGraph, annotateTaintedExports, detectCrossFileFlows, filterFirstPartyNetworkFlows, annotateSinkExports, detectCallbackCrossFileFlows, detectEventEmitterFlows } = require('../scanner/module-graph');
|
|
21
21
|
const { loadCachedIOCs } = require('../ioc/updater.js');
|
|
22
22
|
const { normalizePythonName } = require('../scanner/python.js');
|
|
23
23
|
const { scanPythonSource } = require('../scanner/python-source.js');
|
|
@@ -173,6 +173,10 @@ async function execute(targetPath, options, pythonDeps, warnings) {
|
|
|
173
173
|
// EventEmitter cross-module flow detection
|
|
174
174
|
const emitterFlows = await yieldThen(() => detectEventEmitterFlows(graph, tainted, sinkAnnotations, targetPath));
|
|
175
175
|
crossFileFlows = crossFileFlows.concat(emitterFlows);
|
|
176
|
+
// FP gate (segment A): drop cross_file_dataflow flows whose network sink targets
|
|
177
|
+
// only first-party/local/provider destinations — legit SDK calls, not exfil.
|
|
178
|
+
// Suspicious/unknown/public-IP destinations and exec sinks are kept (ecto stays).
|
|
179
|
+
crossFileFlows = filterFirstPartyNetworkFlows(crossFileFlows, targetPath);
|
|
176
180
|
};
|
|
177
181
|
let graphTimerId;
|
|
178
182
|
const timeout = new Promise((_, reject) => {
|
|
@@ -8,6 +8,7 @@ const { applyFPReductions, applyCompoundBoosts, calculateRiskScore, getSeverityW
|
|
|
8
8
|
const { loadPriorVersionSignatures, computeSignatures, saveCachedSignatures } = require('../scoring/delta-multiplier.js');
|
|
9
9
|
const { annotateConfidenceTiers } = require('../rules/confidence-tiers.js');
|
|
10
10
|
const { buildIntentPairs } = require('../intent-graph.js');
|
|
11
|
+
const { networkDestinationsAllBenign } = require('../sdk-destination.js');
|
|
11
12
|
const { debugLog } = require('../utils.js');
|
|
12
13
|
const { getPackageMetadata } = require('../scanner/npm-registry.js');
|
|
13
14
|
const { checkReleaseZero } = require('../scanner/release-zero.js');
|
|
@@ -351,13 +352,20 @@ async function process(threats, targetPath, options, pythonDeps, warnings, scann
|
|
|
351
352
|
const hasCredFlow = fileThreats.some(t => t.type === 'suspicious_dataflow');
|
|
352
353
|
const alreadyCompound = fileThreats.some(t => t.type === 'detached_credential_exfil');
|
|
353
354
|
if (hasDetached && hasCredFlow && !alreadyCompound) {
|
|
354
|
-
|
|
355
|
-
|
|
356
|
-
|
|
357
|
-
|
|
358
|
-
|
|
359
|
-
|
|
360
|
-
|
|
355
|
+
// FP gate (segment A): skip when the file's network destinations are ALL
|
|
356
|
+
// first-party/local/provider (legit SDK/agent), not exfil. Unknown/suspicious/
|
|
357
|
+
// public-IP host — or unreadable file — keeps it firing (confirmed-benign only).
|
|
358
|
+
let destAllBenign = false;
|
|
359
|
+
try { destAllBenign = networkDestinationsAllBenign(fs.readFileSync(path.join(targetPath, file), 'utf8')); } catch { /* unreadable → not benign */ }
|
|
360
|
+
if (!destAllBenign) {
|
|
361
|
+
deduped.push({
|
|
362
|
+
type: 'detached_credential_exfil',
|
|
363
|
+
severity: 'CRITICAL',
|
|
364
|
+
message: 'Detached process + credential dataflow — background exfiltration (cross-scanner compound).',
|
|
365
|
+
file,
|
|
366
|
+
count: 1
|
|
367
|
+
});
|
|
368
|
+
}
|
|
361
369
|
}
|
|
362
370
|
}
|
|
363
371
|
}
|
|
@@ -1,5 +1,7 @@
|
|
|
1
1
|
'use strict';
|
|
2
2
|
|
|
3
|
+
const { networkDestinationsAllBenign } = require('../../sdk-destination.js');
|
|
4
|
+
|
|
3
5
|
function handlePostWalk(ctx) {
|
|
4
6
|
// SANDWORM_MODE: zlib inflate + base64 decode + eval/Function/Module._compile = obfuscated payload
|
|
5
7
|
if (ctx.hasZlibInflate && ctx.hasBase64Decode && ctx.hasDynamicExec) {
|
|
@@ -322,7 +324,12 @@ function handlePostWalk(ctx) {
|
|
|
322
324
|
const hasSensitiveEnvInFile = ctx.threats.some(t =>
|
|
323
325
|
t.file === ctx.relFile && t.type === 'env_access' && t.severity === 'HIGH'
|
|
324
326
|
);
|
|
325
|
-
|
|
327
|
+
// FP gate (segment A): suppress this credential→network compound when EVERY network
|
|
328
|
+
// destination in the file is first-party/local/provider (e.g. an otel collector on
|
|
329
|
+
// localhost, an SDK POST to its own API). A suspicious/unknown/public-IP host — or no
|
|
330
|
+
// literal host at all — leaves it firing (conservative: confirmed-benign only).
|
|
331
|
+
const destAllBenign = ctx._content ? networkDestinationsAllBenign(ctx._content) : false;
|
|
332
|
+
if (hasDetachedInFile && hasSensitiveEnvInFile && ctx.hasNetworkCallInFile && !destAllBenign) {
|
|
326
333
|
ctx.threats.push({
|
|
327
334
|
type: 'detached_credential_exfil',
|
|
328
335
|
severity: 'CRITICAL',
|
|
@@ -334,7 +341,7 @@ function handlePostWalk(ctx) {
|
|
|
334
341
|
// Audit v3 bypass fix: uncaughtException + env access + network = silent exfiltration
|
|
335
342
|
// Pattern: process.on('uncaughtException', handler) that reads env vars and sends to network.
|
|
336
343
|
// Never legitimate — error handlers don't need to send credentials to external servers.
|
|
337
|
-
if (ctx.hasUncaughtExceptionHandler && hasSensitiveEnvInFile && ctx.hasNetworkCallInFile) {
|
|
344
|
+
if (ctx.hasUncaughtExceptionHandler && hasSensitiveEnvInFile && ctx.hasNetworkCallInFile && !destAllBenign) {
|
|
338
345
|
ctx.threats.push({
|
|
339
346
|
type: 'uncaught_exception_exfil',
|
|
340
347
|
severity: 'CRITICAL',
|
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
'use strict';
|
|
2
2
|
|
|
3
3
|
const path = require('path');
|
|
4
|
-
const { SINK_CALLEE_NAMES, SINK_MEMBER_METHODS, SINK_INSTANCE_METHODS } = require('./constants.js');
|
|
4
|
+
const { SINK_CALLEE_NAMES, SINK_MEMBER_METHODS, SINK_INSTANCE_METHODS, NON_NETWORK_SINK_RECEIVER_ROOTS } = require('./constants.js');
|
|
5
5
|
const {
|
|
6
6
|
parseFile, walkAST, isRequireCall, isModuleExportsAssign,
|
|
7
|
-
getExportName, getFunctionBody, getMemberChain
|
|
7
|
+
getExportName, getFunctionBody, getMemberChain, getReceiverRootName
|
|
8
8
|
} = require('./parse-utils.js');
|
|
9
9
|
|
|
10
10
|
/**
|
|
@@ -87,11 +87,14 @@ function analyzeSinkExports(filePath) {
|
|
|
87
87
|
return;
|
|
88
88
|
}
|
|
89
89
|
}
|
|
90
|
-
// .write(), .send(), .connect()
|
|
90
|
+
// .write(), .send(), .connect() — but not process.*/console.* (local I/O, not network)
|
|
91
91
|
const method = node.callee.property.name || node.callee.property.value;
|
|
92
92
|
if (SINK_INSTANCE_METHODS.has(method)) {
|
|
93
|
-
|
|
94
|
-
|
|
93
|
+
const root = getReceiverRootName(node.callee);
|
|
94
|
+
if (!(root && NON_NETWORK_SINK_RECEIVER_ROOTS.has(root))) {
|
|
95
|
+
found = method + '()';
|
|
96
|
+
return;
|
|
97
|
+
}
|
|
95
98
|
}
|
|
96
99
|
}
|
|
97
100
|
}
|
|
@@ -26,8 +26,17 @@ const SINK_MEMBER_METHODS = new Set([
|
|
|
26
26
|
]);
|
|
27
27
|
const SINK_INSTANCE_METHODS = new Set(['connect', 'write', 'send']);
|
|
28
28
|
|
|
29
|
+
// Receiver roots that make connect/write/send LOCAL I/O or IPC, never external-network
|
|
30
|
+
// exfil: `process.stdout/stderr.write`, `process.send` (child IPC to the parent), and any
|
|
31
|
+
// `console.*`. SINK_INSTANCE_METHODS matches by method name alone, so without this a
|
|
32
|
+
// console/stderr write of a tainted value reads as a cross-file network sink (segment-A FP
|
|
33
|
+
// driver: contextdevkit, amicus). Real socket/ws/req sinks (receivers `socket`/`ws`/`req`/
|
|
34
|
+
// `net.connect()`…) are unaffected. Globals are trusted here as they are everywhere else.
|
|
35
|
+
const NON_NETWORK_SINK_RECEIVER_ROOTS = new Set(['process', 'console']);
|
|
36
|
+
|
|
29
37
|
|
|
30
38
|
module.exports = {
|
|
31
39
|
MAX_GRAPH_NODES, MAX_GRAPH_EDGES, MAX_FLOWS, MAX_TAINT_DEPTH,
|
|
32
|
-
SENSITIVE_MODULES, ACORN_OPTIONS, SINK_CALLEE_NAMES, SINK_MEMBER_METHODS, SINK_INSTANCE_METHODS
|
|
40
|
+
SENSITIVE_MODULES, ACORN_OPTIONS, SINK_CALLEE_NAMES, SINK_MEMBER_METHODS, SINK_INSTANCE_METHODS,
|
|
41
|
+
NON_NETWORK_SINK_RECEIVER_ROOTS
|
|
33
42
|
};
|
|
@@ -1,11 +1,13 @@
|
|
|
1
1
|
'use strict';
|
|
2
2
|
|
|
3
3
|
const path = require('path');
|
|
4
|
+
const fs = require('fs');
|
|
4
5
|
const { debugLog } = require('../../utils');
|
|
5
|
-
const {
|
|
6
|
+
const { networkDestinationsAllBenign } = require('../../sdk-destination.js');
|
|
7
|
+
const { MAX_FLOWS, SINK_CALLEE_NAMES, SINK_MEMBER_METHODS, SINK_INSTANCE_METHODS, NON_NETWORK_SINK_RECEIVER_ROOTS } = require('./constants.js');
|
|
6
8
|
const {
|
|
7
9
|
parseFile, walkAST, isRequireCall, isLocalImport, isModuleExportsAssign,
|
|
8
|
-
getExportName, getMemberChain, resolveLocal
|
|
10
|
+
getExportName, getMemberChain, getReceiverRootName, resolveLocal
|
|
9
11
|
} = require('./parse-utils.js');
|
|
10
12
|
|
|
11
13
|
/**
|
|
@@ -596,7 +598,12 @@ function getSinkName(callNode) {
|
|
|
596
598
|
// instance.connect(), socket.write(), ws.send()
|
|
597
599
|
const method = callee.property.name || callee.property.value;
|
|
598
600
|
if (SINK_INSTANCE_METHODS.has(method)) {
|
|
599
|
-
|
|
601
|
+
// Reject process.*/console.* receivers: process.stdout/stderr.write,
|
|
602
|
+
// process.send (IPC), console.* are local I/O, never external-network exfil.
|
|
603
|
+
const root = getReceiverRootName(callee);
|
|
604
|
+
if (!(root && NON_NETWORK_SINK_RECEIVER_ROOTS.has(root))) {
|
|
605
|
+
return `${method}()`;
|
|
606
|
+
}
|
|
600
607
|
}
|
|
601
608
|
}
|
|
602
609
|
|
|
@@ -954,4 +961,49 @@ function findPipeChainCrossFileFlows(ast, relFile, graph, taintedExports, sinkEx
|
|
|
954
961
|
}
|
|
955
962
|
|
|
956
963
|
|
|
957
|
-
|
|
964
|
+
// A network sink carries a destination host we can judge; exec/command sinks
|
|
965
|
+
// (eval, Function, child_process.*) do not, and are never destination-gated.
|
|
966
|
+
function isNetworkSinkDescriptor(sink) {
|
|
967
|
+
const s = String(sink || '');
|
|
968
|
+
if (/^(eval|Function)\(\)$/.test(s)) return false; // exec sink
|
|
969
|
+
if (/^child_process\./.test(s)) return false; // command sink
|
|
970
|
+
return true; // fetch / http(s).request|get / WebSocket / XMLHttpRequest / connect|write|send
|
|
971
|
+
}
|
|
972
|
+
|
|
973
|
+
/**
|
|
974
|
+
* FP gate (segment A — destination-aware). Drop a cross_file_dataflow whose NETWORK
|
|
975
|
+
* sink targets ONLY benign destinations (loopback/private/reserved IP or a curated
|
|
976
|
+
* provider API) — a legitimate SDK that reads a key and POSTs to its provider is not
|
|
977
|
+
* exfiltration. Untouched (kept CRITICAL): exec/command sinks, and any flow whose sink
|
|
978
|
+
* file references a suspicious/paste host, a public IP, or any unknown domain (so a real
|
|
979
|
+
* exfil like ecto — webhook.site + direct-IP — keeps firing). The package stays visible
|
|
980
|
+
* via its other (lower-severity) signals, the same way intent-graph skips SDK pairs.
|
|
981
|
+
* Rationale + corpus: FPR-segment-A-diagnosis-2026-06-14.md.
|
|
982
|
+
*
|
|
983
|
+
* @param {Array} flows - assembled cross-file flows (main + callback + emitter)
|
|
984
|
+
* @param {string} packagePath - package root, to resolve sink file content
|
|
985
|
+
* @returns {Array} flows with first-party network FPs removed
|
|
986
|
+
*/
|
|
987
|
+
function filterFirstPartyNetworkFlows(flows, packagePath) {
|
|
988
|
+
if (!Array.isArray(flows) || flows.length === 0) return flows;
|
|
989
|
+
const contentCache = new Map();
|
|
990
|
+
const kept = [];
|
|
991
|
+
for (const flow of flows) {
|
|
992
|
+
if (flow && flow.type === 'cross_file_dataflow' && flow.sinkFile && isNetworkSinkDescriptor(flow.sink)) {
|
|
993
|
+
let content = contentCache.get(flow.sinkFile);
|
|
994
|
+
if (content === undefined) {
|
|
995
|
+
try { content = fs.readFileSync(path.resolve(packagePath, flow.sinkFile), 'utf8'); }
|
|
996
|
+
catch { content = ''; }
|
|
997
|
+
contentCache.set(flow.sinkFile, content);
|
|
998
|
+
}
|
|
999
|
+
if (content && networkDestinationsAllBenign(content)) {
|
|
1000
|
+
debugLog(`[MODULE-GRAPH] cross_file_dataflow suppressed (first-party/local dest): ${flow.sourceFile} -> ${flow.sink} in ${flow.sinkFile}`);
|
|
1001
|
+
continue; // first-party/local network destination → FP, drop
|
|
1002
|
+
}
|
|
1003
|
+
}
|
|
1004
|
+
kept.push(flow);
|
|
1005
|
+
}
|
|
1006
|
+
return kept;
|
|
1007
|
+
}
|
|
1008
|
+
|
|
1009
|
+
module.exports = { detectCrossFileFlows, expandTaintThroughReexports, collectImportTaint, propagateLocalTaint, getSinkName, findTaintedArgument, isNetworkSinkDescriptor, filterFirstPartyNetworkFlows };
|
|
@@ -4,13 +4,13 @@ const { MAX_GRAPH_NODES, MAX_GRAPH_EDGES, MAX_FLOWS, MAX_TAINT_DEPTH } = require
|
|
|
4
4
|
const { parseFile, resolveLocal, isLocalImport, toRel, isFileExists } = require('./parse-utils.js');
|
|
5
5
|
const { buildModuleGraph, extractLocalImports, tryResolveConcatRequire } = require('./build-graph.js');
|
|
6
6
|
const { annotateTaintedExports } = require('./annotate-tainted.js');
|
|
7
|
-
const { detectCrossFileFlows } = require('./detect-cross-file.js');
|
|
7
|
+
const { detectCrossFileFlows, filterFirstPartyNetworkFlows } = require('./detect-cross-file.js');
|
|
8
8
|
const { annotateSinkExports } = require('./annotate-sinks.js');
|
|
9
9
|
const { detectCallbackCrossFileFlows } = require('./detect-callback-flows.js');
|
|
10
10
|
const { detectEventEmitterFlows } = require('./detect-event-flows.js');
|
|
11
11
|
|
|
12
12
|
module.exports = {
|
|
13
|
-
buildModuleGraph, annotateTaintedExports, detectCrossFileFlows,
|
|
13
|
+
buildModuleGraph, annotateTaintedExports, detectCrossFileFlows, filterFirstPartyNetworkFlows,
|
|
14
14
|
annotateSinkExports, detectCallbackCrossFileFlows, detectEventEmitterFlows,
|
|
15
15
|
resolveLocal, extractLocalImports, parseFile, isLocalImport, toRel, isFileExists,
|
|
16
16
|
tryResolveConcatRequire,
|
|
@@ -107,6 +107,18 @@ function getMemberChain(node, depth) {
|
|
|
107
107
|
return '';
|
|
108
108
|
}
|
|
109
109
|
|
|
110
|
+
// Root identifier of a call's receiver, e.g. `process` for (process.stdout).write(),
|
|
111
|
+
// `process` for process.send(), `console` for console.error(), `sender` for sender.send().
|
|
112
|
+
// Returns null when the receiver root is not a plain Identifier (e.g. this.x.write(),
|
|
113
|
+
// foo().bar()). Used to reject local-IO/IPC receivers (process/console) from the
|
|
114
|
+
// write/send/connect instance-method sink set, which matches by method name alone.
|
|
115
|
+
function getReceiverRootName(callee) {
|
|
116
|
+
if (!callee || callee.type !== 'MemberExpression') return null;
|
|
117
|
+
let obj = callee.object;
|
|
118
|
+
while (obj && obj.type === 'MemberExpression') obj = obj.object;
|
|
119
|
+
return obj && obj.type === 'Identifier' ? obj.name : null;
|
|
120
|
+
}
|
|
121
|
+
|
|
110
122
|
function extractLiteralArg(args) {
|
|
111
123
|
if (!args || args.length === 0) return '';
|
|
112
124
|
const first = args[0];
|
|
@@ -136,6 +148,6 @@ function toRel(abs, packagePath) {
|
|
|
136
148
|
|
|
137
149
|
module.exports = {
|
|
138
150
|
parseFile, walkAST, isRequireCall, isLocalImport, isModuleExportsAssign,
|
|
139
|
-
getExportName, getFunctionBody, getMemberChain, extractLiteralArg,
|
|
151
|
+
getExportName, getFunctionBody, getMemberChain, getReceiverRootName, extractLiteralArg,
|
|
140
152
|
resolveLocal, isFileExists, toRel
|
|
141
153
|
};
|