CVE-2024-25117: Phenx/php svg lib Php-svg-lib Lacks Path Validation On Vulnerability

MODERATE
Advisory Identifier: GHSA-f3qr-qr4x-j273
Share
CVSS 6.8
CWE-502 CWE-610 CWE-73
Published
Advisory Summary

CVE-2024-25117 is a MODERATE (CVSS 6.8) php-svg-lib lacks path validation on vulnerability (CWE-502: Deserialization of Untrusted Data) affecting Phenx/php svg lib libraries in the Packagist ecosystem. The issue involves php-svg-lib lacks path validation on font through svg inline styles. Remediation is available by upgrading affected packages to version 0.5.2.

CVE-2024-25117 at a Glance

CVE Identifier CVE-2024-25117
GHSA Identifier GHSA-f3qr-qr4x-j273
Severity MODERATE
CVSS v3.1 Base Score 6.8
CWE Classification CWE-502: Deserialization of Untrusted Data
Affected Product Phenx/php svg lib
Package Ecosystem Packagist
Published Date
Last Updated

Security Impact

CVSS: 6.8

The security impact metrics below are parsed directly from the CVSS v3.1 vector string (CVSS:3.1/AV:L/AC:L/PR:N/UI:N/S:C/C:L/I:L/A:L).

Attack Vector Local (AV:L)
Attack Complexity Low (AC:L)
Privileges Required None (PR:N)
User Interaction None (UI:N)
Scope Changed (S:C)
Confidentiality Low (C:L)
Integrity Low (I:L)
Availability Low (A:L)

How to Fix CVE-2024-25117

To remediate this vulnerability, upgrade all affected Phenx/php svg lib dependencies to a patched release. Both direct and transitive dependencies should be checked, as frameworks or security libraries may pull in vulnerable versions transitively.

phenx/php-svg-lib fixed: 0.5.2 
Upgrade phenx/php-svg-lib to 0.5.2
Verify Direct and Transitive Dependencies

Run the following command in your project directory to verify that all resolved dependencies use the patched release: 

inspect dependencies for phenx/php-svg-lib

How to Check if You Are Affected

To determine whether your application is affected by CVE-2024-25117, inspect your project dependencies for Phenx/php svg lib:

inspect dependencies for phenx/php-svg-lib

Analyze the terminal output to locate any version of Phenx/php svg lib in your dependency tree. If any resolved version matches the vulnerable ranges listed in the table below (for example, versions earlier than 0.5.2), your application is affected.

Note: Verify declared build manifests and deployed runtime container environments.

Technical Details

Vulnerability Mechanism & Technical Analysis

This vulnerability is classified under CWE-502: Deserialization of Untrusted Data, where the product deserializes untrusted data without sufficiently ensuring that the resulting data will be valid. In Phenx/php svg lib, software state validation or input boundaries are circumvented during processing. Review the official source advisory statement below for specific affected components, classes, and reproduction details.

Source Advisory Statement

### Summary php-svg-lib fails to validate that font-family doesn't contain a PHAR url, which might leads to RCE on PHP < 8.0, and doesn't validate if external references are allowed. This might leads to bypass of restrictions or RCE on projects that are using it, if they do not strictly revalidate the fontName that is passed by php-svg-lib. ### Details The Style::fromAttributes(), or the Style::parseCssStyle() should check the content of the `font-family` and prevents it to use a PHAR url, to avoid passing an invalid and dangerous `fontName` value to other libraries. The same check as done in the Style::fromStyleSheets might be reused : ``` if ( \array_key_exists("font-family", $styles) && ( \strtolower(\substr($this->href, 0, 7)) === "phar://" || ($this->document->allowExternalReferences === false && \strtolower(\substr($this->href, 0, 5)) !== "data:") ) ) { unset($style["font-family"]); } ``` ### PoC Parsing the following SVG : ``` <?xml version="1.0" encoding="UTF-8" standalone="no"?> <svg xmlns:svg="http://www.w3.org/2000/svg" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="200" height="200"> <text x="20" y="35" style="color:red;font-family:phar:///path/to/whatever.phar/blaklis;">My</text> </svg> ``` will pass the `phar:///path/to/whatever.phar/blaklis` as `$family` in `SurfaceCpdf::setFont`, which is then passed to the canvas `selectFont` as a `$fontName`. ### Impact Libraries using this library as a dependency might be vulnerable to some bypass of restrictions, or even RCE, if they do not double check the value of the `fontName` that is passed by php-svg-lib

CVE-2024-25117 Affected Phenx/php svg lib Versions

1 component(s)
Fixed Versions:
0.5.2
Product / Package Ecosystem Affected Versions Fixed Version
phenx/php-svg-lib
 Packagist
< 0.5.2
0.5.2

References

GitHub Advisory / Wiki ADVISORY
https://github.com/dompdf/php-svg-lib/security/advisories/GHSA-f3qr-qr4x-j273
NVD — CVE-2024-25117 NVD
https://nvd.nist.gov/vuln/detail/CVE-2024-25117
Security Fix Commit FIX COMMIT
https://github.com/dompdf/php-svg-lib/commit/732faa9fb4309221e2bd9b2fda5de44f947133aa
Security Fix Commit FIX COMMIT
https://github.com/dompdf/php-svg-lib/commit/8ffcc41bbde39f09f94b9760768086f12bbdce42
Phenx/php svg lib Source Repository REPOSITORY
https://github.com/dompdf/php-svg-lib

Frequently Asked Questions

What is CVE-2024-25117?
CVE-2024-25117 is a security vulnerability in Phenx/php svg lib involving php-svg-lib lacks path validation on font through SVG inline styles under CWE-502: Deserialization of Untrusted Data.
Which Phenx/php svg lib versions are affected?
Affected versions include releases prior to 0.5.2 across affected packages: phenx/php-svg-lib.
What version fixes CVE-2024-25117?
Upgrading to 0.5.2 (or the latest patched release for your respective branch) resolves the vulnerability.
Is phenx/php-svg-lib affected?
Yes, phenx/php-svg-lib is affected in releases prior to 0.5.2. Upgrade your dependency to 0.5.2 or later.
What is the CVSS score of CVE-2024-25117?
CVE-2024-25117 has a CVSS v3.1 base score of 6.8 (MODERATE) with vector string CVSS:3.1/AV:L/AC:L/PR:N/UI:N/S:C/C:L/I:L/A:L.
What CWE is associated with CVE-2024-25117?
CVE-2024-25117 is classified under CWE-502: Deserialization of Untrusted Data (The product deserializes untrusted data without sufficiently ensuring that the resulting data will be valid.).
How can I check Packagist dependencies for CVE-2024-25117?
Run the dependency tree command: 'inspect dependencies for phenx/php-svg-lib'. This command lists direct and transitive dependencies in your build configuration so you can verify if a vulnerable version is resolved.
Machine-readable OSV JSON OSV v1.5 Schema
Normalized schema output
View Raw JSON 
{
  "schema_version" : "1.4.0",
  "id" : "GHSA-f3qr-qr4x-j273",
  "modified" : "2024-02-21T19:33:13Z",
  "published" : "2024-02-21T18:04:16Z",
  "aliases" : [ "CVE-2024-25117" ],
  "summary" : "php-svg-lib lacks path validation on font through SVG inline styles ",
  "details" : "### Summary\nphp-svg-lib fails to validate that font-family doesn't contain a PHAR url, which might leads to RCE on PHP < 8.0, and doesn't validate if external references are allowed. This might leads to bypass of restrictions or RCE on projects that are using it, if they do not strictly revalidate the fontName that is passed by php-svg-lib.\n\n### Details\nThe Style::fromAttributes(), or the Style::parseCssStyle() should check the content of the `font-family` and prevents it to use a PHAR url, to avoid passing an invalid and dangerous `fontName` value to other libraries. The same check as done in the Style::fromStyleSheets might be reused : \n\n```\n                if (\n                    \\array_key_exists(\"font-family\", $styles)\n                    && (\n                        \\strtolower(\\substr($this->href, 0, 7)) === \"phar://\"\n                        || ($this->document->allowExternalReferences === false && \\strtolower(\\substr($this->href, 0, 5)) !== \"data:\")\n                    )\n                ) {\n                    unset($style[\"font-family\"]);\n                }\n```\n\n### PoC \n\nParsing the following SVG : \n\n```\n<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"no\"?>\n<svg xmlns:svg=\"http://www.w3.org/2000/svg\" xmlns=\"http://www.w3.org/2000/svg\" xmlns:xlink=\"http://www.w3.org/1999/xlink\" width=\"200\" height=\"200\">\n    <text x=\"20\" y=\"35\" style=\"color:red;font-family:phar:///path/to/whatever.phar/blaklis;\">My</text>\n</svg>\n```\n\nwill pass the `phar:///path/to/whatever.phar/blaklis` as `$family` in `SurfaceCpdf::setFont`, which is then passed to the canvas `selectFont` as a `$fontName`.\n\n### Impact\nLibraries using this library as a dependency might be vulnerable to some bypass of restrictions, or even RCE, if they do not double check the value of the `fontName` that is passed by php-svg-lib",
  "severity" : [ {
    "type" : "CVSS_V3",
    "score" : "CVSS:3.1/AV:L/AC:L/PR:N/UI:N/S:C/C:L/I:L/A:L"
  } ],
  "affected" : [ {
    "package" : {
      "ecosystem" : "Packagist",
      "name" : "phenx/php-svg-lib"
    },
    "ranges" : [ {
      "type" : "ECOSYSTEM",
      "events" : [ {
        "introduced" : "0"
      }, {
        "fixed" : "0.5.2"
      } ]
    } ]
  } ],
  "references" : [ {
    "type" : "WEB",
    "url" : "https://github.com/dompdf/php-svg-lib/security/advisories/GHSA-f3qr-qr4x-j273"
  }, {
    "type" : "ADVISORY",
    "url" : "https://nvd.nist.gov/vuln/detail/CVE-2024-25117"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/dompdf/php-svg-lib/commit/732faa9fb4309221e2bd9b2fda5de44f947133aa"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/dompdf/php-svg-lib/commit/8ffcc41bbde39f09f94b9760768086f12bbdce42"
  }, {
    "type" : "PACKAGE",
    "url" : "https://github.com/dompdf/php-svg-lib"
  } ],
  "database_specific" : {
    "cwe_ids" : [ "CWE-502", "CWE-610", "CWE-73" ],
    "severity" : "MODERATE",
    "github_reviewed" : true,
    "github_reviewed_at" : "2024-02-21T18:04:16Z",
    "nvd_published_at" : "2024-02-21T17:15:09Z"
  }
}