cognium-dev 3.178.0 → 3.180.0

package/dist/cli.js

@@ -11683,12 +11683,20 @@ var DEFAULT_SINKS = [
   { method: "format", class: "String", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
   { method: "format", class: "Formatter", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
   { method: "printf", class: "System.out", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
+  { method: "format", class: "MessageFormat", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
+  { method: "printf", class: "PrintStream", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
+  { method: "format", class: "PrintStream", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
+  { method: "printf", class: "PrintWriter", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
+  { method: "format", class: "PrintWriter", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["java"] },
   { method: "printf", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [0], languages: ["python"] },
   { method: "fprintf", type: "format_string", cwe: "CWE-134", severity: "high", arg_positions: [1], languages: ["python"] },
   { method: "Sprintf", class: "fmt", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [0], languages: ["go"] },
   { method: "Printf", class: "fmt", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [0], languages: ["go"] },
   { method: "Errorf", class: "fmt", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [0], languages: ["go"] },
   { method: "Fprintf", class: "fmt", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [1], languages: ["go"] },
+  { method: "Printf", class: "log", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [0], languages: ["go"] },
+  { method: "Fatalf", class: "log", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [0], languages: ["go"] },
+  { method: "Panicf", class: "log", type: "format_string", cwe: "CWE-134", severity: "medium", arg_positions: [0], languages: ["go"] },
   { method: "setHeader", type: "crlf", cwe: "CWE-113", severity: "medium", arg_positions: [1], languages: ["javascript", "typescript"] },
   { method: "writeHead", type: "crlf", cwe: "CWE-113", severity: "medium", arg_positions: [2], languages: ["javascript", "typescript"] },
   { method: "cookie", type: "crlf", cwe: "CWE-113", severity: "medium", arg_positions: [1], languages: ["javascript", "typescript"] },
@@ -13680,6 +13688,11 @@ function calculateSinkConfidence(call, pattern) {
   if (pattern.class && call.receiver) {
     if (receiverMightBeClass(call.receiver, pattern.class)) {
       confidence += 0.1;
+      const receiverLc = call.receiver.toLowerCase();
+      const classLc = pattern.class.toLowerCase();
+      if (call.receiver === pattern.class || receiverLc === classLc) {
+        confidence += 0.05;
+      }
     }
   }
   if (pattern.severity === "critical") {
@@ -32655,6 +32668,130 @@ function resolveFastjsonFromPom(pomXml) {
   }
   return null;
 }
+function resolveFastjsonFromGradle(buildGradle) {
+  if (!buildGradle)
+    return null;
+  const directRe = /['"(]\s*com\.alibaba:fastjson:([^'"$\s)]+)\s*['")]/;
+  const direct = buildGradle.match(directRe);
+  if (direct) {
+    const version2 = direct[1];
+    return { version: version2, noneAutotype: /_noneautotype/i.test(version2) };
+  }
+  const constraintsBlockRe = /['"(]\s*com\.alibaba:fastjson\s*['")]\s*\)?\s*\{[\s\S]*?\bversion\s*\{[\s\S]*?\b(?:strictly|require|prefer)\s+['"]([^'"\s]+)['"]/;
+  const constraintsBlock = buildGradle.match(constraintsBlockRe);
+  if (constraintsBlock) {
+    const version2 = constraintsBlock[1];
+    return { version: version2, noneAutotype: /_noneautotype/i.test(version2) };
+  }
+  const propRefRe = /['"(]\s*com\.alibaba:fastjson:\$\{?([A-Za-z_][A-Za-z0-9_]*)\}?\s*['")]/;
+  const propRef = buildGradle.match(propRefRe);
+  if (!propRef)
+    return null;
+  const propName = propRef[1];
+  const defRe = new RegExp(`\\b${propName}\\s*[=:]\\s*['"]([^'"\\s]+)['"]`);
+  const def = buildGradle.match(defRe);
+  if (!def)
+    return null;
+  const version = def[1];
+  return { version, noneAutotype: /_noneautotype/i.test(version) };
+}
+function resolveFastjsonFromGradleCatalog(buildGradle, libsVersionsToml) {
+  if (!buildGradle || !libsVersionsToml)
+    return null;
+  const librariesMatch = libsVersionsToml.match(/\[libraries\]([\s\S]*?)(?=\n\[|$)/);
+  if (!librariesMatch)
+    return null;
+  const librariesBlock = librariesMatch[1];
+  const moduleForm = librariesBlock.match(/^\s*([\w.-]+)\s*=\s*\{[^}]*\bmodule\s*=\s*"com\.alibaba:fastjson"[^}]*\}/m);
+  const groupNameForm = librariesBlock.match(/^\s*([\w.-]+)\s*=\s*\{[^}]*\bgroup\s*=\s*"com\.alibaba"[^}]*\bname\s*=\s*"fastjson"[^}]*\}/m);
+  const entry = moduleForm ?? groupNameForm;
+  if (!entry)
+    return null;
+  const alias = entry[1];
+  const entryBody = entry[0];
+  const aliasEsc = alias.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  const aliasRefRe = new RegExp(`\\blibs\\s*\\.\\s*${aliasEsc}\\b`);
+  if (!aliasRefRe.test(buildGradle))
+    return null;
+  const inlineVersionM = entryBody.match(/\bversion\s*=\s*"([^"$][^"]*)"/);
+  if (inlineVersionM) {
+    const version2 = inlineVersionM[1];
+    return { version: version2, noneAutotype: /_noneautotype/i.test(version2) };
+  }
+  const versionRefM = entryBody.match(/\bversion\.ref\s*=\s*"([^"]+)"/);
+  if (!versionRefM)
+    return null;
+  const versionAlias = versionRefM[1];
+  const versionsMatch = libsVersionsToml.match(/\[versions\]([\s\S]*?)(?=\n\[|$)/);
+  if (!versionsMatch)
+    return null;
+  const versionsBlock = versionsMatch[1];
+  const versionAliasEsc = versionAlias.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  const versionLineRe = new RegExp(`^\\s*${versionAliasEsc}\\s*=\\s*"([^"]+)"`, "m");
+  const versionLine = versionsBlock.match(versionLineRe);
+  if (!versionLine)
+    return null;
+  const version = versionLine[1];
+  return { version, noneAutotype: /_noneautotype/i.test(version) };
+}
+function resolvePyYamlFromRequirements(requirementsTxt) {
+  if (!requirementsTxt)
+    return null;
+  for (const rawLine of requirementsTxt.split(/\r?\n/)) {
+    const line = rawLine.replace(/#.*$/, "").trim();
+    if (!line)
+      continue;
+    if (line.startsWith("-"))
+      continue;
+    const m = line.match(/^(pyyaml)\s*(==|>=|~=|>|===)\s*(\d+(?:\.\d+)*)/i);
+    if (!m)
+      continue;
+    const version = m[3];
+    return { version, safeByDefault: isPyYamlVersionSafeByDefault(version) };
+  }
+  return null;
+}
+function resolvePyYamlFromPyproject(pyprojectToml) {
+  if (!pyprojectToml)
+    return null;
+  const poetryLine = pyprojectToml.match(/^\s*(pyyaml)\s*=\s*"([\^~=<>]*)(\d+(?:\.\d+)*)"/im);
+  if (poetryLine) {
+    const version = poetryLine[3];
+    return { version, safeByDefault: isPyYamlVersionSafeByDefault(version) };
+  }
+  const poetryTable = pyprojectToml.match(/^\s*(pyyaml)\s*=\s*\{[^}]*\bversion\s*=\s*"([\^~=<>]*)(\d+(?:\.\d+)*)"/im);
+  if (poetryTable) {
+    const version = poetryTable[3];
+    return { version, safeByDefault: isPyYamlVersionSafeByDefault(version) };
+  }
+  const pep621 = pyprojectToml.match(/"(pyyaml)\s*(==|>=|~=|>|===)\s*(\d+(?:\.\d+)*)/i);
+  if (pep621) {
+    const version = pep621[3];
+    return { version, safeByDefault: isPyYamlVersionSafeByDefault(version) };
+  }
+  return null;
+}
+function isPyYamlVersionSafeByDefault(version) {
+  const parts2 = version.split(".").map((s) => Number.parseInt(s, 10));
+  if (parts2.length === 0 || !Number.isFinite(parts2[0]))
+    return false;
+  return parts2[0] >= 6;
+}
+function fileHasUnsafePyYamlLoader(sourceLines, sinkLine) {
+  const start2 = Math.max(0, sinkLine - 1);
+  const end = Math.min(sourceLines.length, start2 + 10);
+  const unsafeRe = /\bLoader\s*=\s*(?:yaml\s*\.\s*)?(?:Loader|UnsafeLoader|FullLoader)\b/;
+  for (let i2 = start2;i2 < end; i2++) {
+    const ln = sourceLines[i2] ?? "";
+    if (unsafeRe.test(ln))
+      return true;
+    const stripped = ln.trim();
+    if (i2 > start2 && (stripped === ")" || stripped.endsWith(")"))) {
+      return false;
+    }
+  }
+  return false;
+}
 function fileReenablesFastjsonAutotype(source) {
   if (!source)
     return false;
@@ -32688,6 +32825,8 @@ var JACKSON_METHODS = new Set(["readValue", "convertValue", "treeToValue"]);
 var JACKSON_CLASSES = new Set(["ObjectMapper", "ObjectReader"]);
 var SNAKEYAML_METHODS = new Set(["load", "loadAs", "loadAll"]);
 var SNAKEYAML_CLASSES = new Set(["Yaml"]);
+var PYYAML_METHODS = new Set(["load"]);
+var PYYAML_CLASSES = new Set(["yaml"]);
 
 class DeserializationSafetyGatePass {
   dependencyContext;
@@ -32698,18 +32837,32 @@ class DeserializationSafetyGatePass {
   }
   run(ctx) {
     const { graph, language, code } = ctx;
-    if (language !== "java") {
-      return { droppedFastjson: 0, droppedJackson: 0, droppedSnakeYaml: 0 };
+    if (language !== "java" && language !== "python") {
+      return { droppedFastjson: 0, droppedJackson: 0, droppedSnakeYaml: 0, droppedPyYaml: 0 };
     }
     const sinks = ctx.hasResult("sink-filter") ? ctx.getResult("sink-filter").sinks : graph.ir.taint.sinks;
     const pomXml = this.dependencyContext?.java?.pomXml;
-    const fastjson = pomXml ? resolveFastjsonFromPom(pomXml) : null;
+    const buildGradle = this.dependencyContext?.java?.buildGradle;
+    const libsVersionsToml = this.dependencyContext?.java?.libsVersionsToml;
+    const fastjson = (pomXml ? resolveFastjsonFromPom(pomXml) : null) ?? (buildGradle ? resolveFastjsonFromGradle(buildGradle) : null) ?? (buildGradle && libsVersionsToml ? resolveFastjsonFromGradleCatalog(buildGradle, libsVersionsToml) : null);
     const fastjsonHardened = fastjson?.noneAutotype === true && !fileReenablesFastjsonAutotype(code);
     const jacksonSafe = !fileEnablesJacksonPolymorphism(code);
     const snakeYamlSafe = fileConfiguresSnakeYamlSafely(code);
+    const requirementsTxt = this.dependencyContext?.python?.requirementsTxt;
+    const pyprojectToml = this.dependencyContext?.python?.pyprojectToml;
+    const pyYaml = (requirementsTxt ? resolvePyYamlFromRequirements(requirementsTxt) : null) ?? (pyprojectToml ? resolvePyYamlFromPyproject(pyprojectToml) : null);
+    const pyYamlSafeByDefault = pyYaml?.safeByDefault === true;
+    let sourceLines = null;
+    const getSourceLines = () => {
+      if (sourceLines === null)
+        sourceLines = code.split(`
+`);
+      return sourceLines;
+    };
     let droppedFastjson = 0;
     let droppedJackson = 0;
     let droppedSnakeYaml = 0;
+    let droppedPyYaml = 0;
     const kept = sinks.filter((sink) => {
       if (sink.type !== "deserialization")
         return true;
@@ -32727,14 +32880,18 @@ class DeserializationSafetyGatePass {
         droppedSnakeYaml++;
         return false;
       }
+      if (language === "python" && pyYamlSafeByDefault && PYYAML_METHODS.has(sink.method) && (sink.class === undefined || PYYAML_CLASSES.has(sink.class)) && !fileHasUnsafePyYamlLoader(getSourceLines(), sink.line)) {
+        droppedPyYaml++;
+        return false;
+      }
       return true;
     });
-    const totalDropped = droppedFastjson + droppedJackson + droppedSnakeYaml;
+    const totalDropped = droppedFastjson + droppedJackson + droppedSnakeYaml + droppedPyYaml;
     if (totalDropped > 0) {
       sinks.length = 0;
       sinks.push(...kept);
     }
-    return { droppedFastjson, droppedJackson, droppedSnakeYaml };
+    return { droppedFastjson, droppedJackson, droppedSnakeYaml, droppedPyYaml };
   }
 }
 
@@ -39624,6 +39781,47 @@ class ScanSecretsPass {
   }
 }
 
+// ../circle-ir/dist/analysis/passes/python-receiver-taint-format-pass.js
+class PythonReceiverTaintFormatPass {
+  name = "python-receiver-taint-format";
+  category = "security";
+  run(ctx) {
+    if (ctx.language !== "python") {
+      return { findingsEmitted: 0 };
+    }
+    const { calls, meta } = ctx.graph.ir;
+    const constProp = ctx.getResult("constant-propagation");
+    const tainted = constProp.tainted;
+    let count = 0;
+    for (const call of calls) {
+      if (call.method_name !== "format")
+        continue;
+      const receiver = call.receiver;
+      if (!receiver)
+        continue;
+      if (!/^[A-Za-z_][A-Za-z0-9_]*$/.test(receiver))
+        continue;
+      const scopedName = call.in_method ? `${call.in_method}:${receiver}` : receiver;
+      if (!tainted.has(receiver) && !tainted.has(scopedName))
+        continue;
+      ctx.addFinding({
+        id: `format_string-${meta.file}-${call.location.line}-${receiver}`,
+        pass: this.name,
+        category: "security",
+        rule_id: "format_string",
+        cwe: "CWE-134",
+        severity: "medium",
+        level: "warning",
+        message: `Format-string vulnerability: the format template \`${receiver}\` ` + `is tainted (traces back to a taint source). Attacker can leak ` + `object attributes via \`{obj.__class__.__mro__[…]}\` chains or ` + `crash the process via unbounded specifiers. Use a literal ` + `format string and pass user input as an argument instead.`,
+        file: meta.file,
+        line: call.location.line
+      });
+      count++;
+    }
+    return { findingsEmitted: count };
+  }
+}
+
 // ../circle-ir/dist/analysis/passes/spring4shell-pass.js
 var CONTROLLER_ANNOTATIONS = new Set([
   "Controller",
@@ -44295,6 +44493,8 @@ async function analyze(code, filePath, language, options = {}) {
       pipeline.add(new LibraryProfileCwe22PathGatePass);
     if (!disabledPasses.has("scan-secrets"))
       pipeline.add(new ScanSecretsPass);
+    if (!disabledPasses.has("python-receiver-taint-format"))
+      pipeline.add(new PythonReceiverTaintFormatPass);
     if (!disabledPasses.has("dead-code"))
       pipeline.add(new DeadCodePass);
     if (!disabledPasses.has("missing-await"))
@@ -45224,7 +45424,7 @@ var colors = {
 };
 
 // src/version.ts
-var version = "3.178.0";
+var version = "3.180.0";
 
 // src/formatters.ts
 var LIBRARY_API_SURFACE_TAG2 = "library-api-surface:caller-responsibility";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cognium-dev",
-  "version": "3.178.0",
+  "version": "3.180.0",
   "description": "Static Application Security Testing CLI for detecting security vulnerabilities via taint tracking",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -66,7 +66,7 @@
   },
   "dependencies": {
     "@cognium/project-profile-detect": "^1.1.0",
-    "circle-ir": "^3.178.0"
+    "circle-ir": "^3.180.0"
   },
   "devDependencies": {
     "@types/node": "^25.5.0",