GHSA-4X9G-VW65-VVF9

Vulnerability from github – Published: 2026-08-14 19:23 – Updated: 2026-08-14 19:23
VLAI
Summary
Grav: Unauthenticated denial of service via unbounded image derivative dimensions
Details

Summary

An unauthenticated visitor exhausts server memory and CPU by requesting an image with oversized resize dimensions. One request drives a worker to several gigabytes of RAM and tens of seconds of CPU. A few concurrent requests take the host down.

Details

Grav::fallbackUrl() (system/src/Grav/Common/Grav.php:800-804) loops over every query parameter and, when the name matches ImageMedium::$magic_actions, calls that method on the medium with the comma-split value as arguments:

foreach ($uri->query(null, true) as $action => $params) {
    if (in_array($action, ImageMedium::$magic_actions, true)) {
        call_user_func_array([&$medium, $action], explode(',', $params));
    }
}

forceResize runs with force=true, so it sets the output size to the attacker's values with no clamp against the source or any ceiling. The getgrav/image GD adapter then calls imagecreatetruecolor($w, $h). libgd allocates that buffer outside PHP's emalloc, so memory_limit does not cap it. Grav exposes no system.images.max_width/max_height setting.

PoC

Any page that serves an image works. With a 200x150 source image:

GET /home/test.png?forceResize=20000,20000

Measured on PHP 8.4.21 with memory_limit=128M:

  • peak worker RSS 3,109 MB
  • 21.9 s CPU
  • HTTP 200, 1.6 MB response

8000x8000 already needs ~244 MB. The cache key includes the dimensions, so varying them forces fresh work on every request.

Impact

Unauthenticated denial of service against any Grav site that serves images. No account, plugin, or non-default config required.

Fix

Clamp the request-derived dimensions before dispatch, behind a configurable cap. The image library is the wrong layer; bound the arguments at the request boundary.

--- a/system/src/Grav/Common/Grav.php
+++ b/system/src/Grav/Common/Grav.php
@@ public function fallbackUrl($path)
                 foreach ($uri->query(null, true) as $action => $params) {
                     if (in_array($action, ImageMedium::$magic_actions, true)) {
-                        call_user_func_array([&$medium, $action], explode(',', $params));
+                        $args = explode(',', $params);
+                        $max = (int) $config->get('system.images.max_dimension', 8000);
+                        if ($max > 0
+                            && in_array($action, ['resize', 'forceResize', 'cropResize', 'cropZoom', 'zoomCrop', 'crop'], true)) {
+                            foreach ($args as $a) {
+                                if (is_numeric($a) && (int) $a > $max) {
+                                    return false; // reject oversized derivative request
+                                }
+                            }
+                        }
+                        call_user_func_array([&$medium, $action], $args);
                     }
                 }

Document system.images.max_dimension (default 8000) so operators can tune it. A total-pixel ceiling (width * height) is a stricter alternative.

Show details on source website

{
  "affected": [
    {
      "package": {
        "ecosystem": "Packagist",
        "name": "getgrav/grav"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "2.0.0-beta.1"
            },
            {
              "fixed": "2.0.0-rc.8"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    },
    {
      "package": {
        "ecosystem": "Packagist",
        "name": "getgrav/grav"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "0"
            },
            {
              "fixed": "1.7.53"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    }
  ],
  "aliases": [
    "CVE-2026-53653"
  ],
  "database_specific": {
    "cwe_ids": [
      "CWE-770"
    ],
    "github_reviewed": true,
    "github_reviewed_at": "2026-08-14T19:23:38Z",
    "nvd_published_at": "2026-07-10T17:16:57Z",
    "severity": "HIGH"
  },
  "details": "### Summary\nAn unauthenticated visitor exhausts server memory and CPU by requesting an image with oversized resize dimensions. One request drives a worker to several gigabytes of RAM and tens of seconds of CPU. A few concurrent requests take the host down.\n\n### Details\n`Grav::fallbackUrl()` (system/src/Grav/Common/Grav.php:800-804) loops over every query parameter and, when the name matches `ImageMedium::$magic_actions`, calls that method on the medium with the comma-split value as arguments:\n\n```php\nforeach ($uri-\u003equery(null, true) as $action =\u003e $params) {\n    if (in_array($action, ImageMedium::$magic_actions, true)) {\n        call_user_func_array([\u0026$medium, $action], explode(\u0027,\u0027, $params));\n    }\n}\n```\n\n`forceResize` runs with `force=true`, so it sets the output size to the attacker\u0027s values with no clamp against the source or any ceiling. The `getgrav/image` GD adapter then calls `imagecreatetruecolor($w, $h)`. libgd allocates that buffer outside PHP\u0027s `emalloc`, so `memory_limit` does not cap it. Grav exposes no `system.images.max_width`/`max_height` setting.\n\n### PoC\nAny page that serves an image works. With a 200x150 source image:\n\n```\nGET /home/test.png?forceResize=20000,20000\n```\n\nMeasured on PHP 8.4.21 with `memory_limit=128M`:\n\n- peak worker RSS 3,109 MB\n- 21.9 s CPU\n- HTTP 200, 1.6 MB response\n\n`8000x8000` already needs ~244 MB. The cache key includes the dimensions, so varying them forces fresh work on every request.\n\n### Impact\nUnauthenticated denial of service against any Grav site that serves images. No account, plugin, or non-default config required.\n\n## Fix\nClamp the request-derived dimensions before dispatch, behind a configurable cap. The image library is the wrong layer; bound the arguments at the request boundary.\n\n```diff\n--- a/system/src/Grav/Common/Grav.php\n+++ b/system/src/Grav/Common/Grav.php\n@@ public function fallbackUrl($path)\n                 foreach ($uri-\u003equery(null, true) as $action =\u003e $params) {\n                     if (in_array($action, ImageMedium::$magic_actions, true)) {\n-                        call_user_func_array([\u0026$medium, $action], explode(\u0027,\u0027, $params));\n+                        $args = explode(\u0027,\u0027, $params);\n+                        $max = (int) $config-\u003eget(\u0027system.images.max_dimension\u0027, 8000);\n+                        if ($max \u003e 0\n+                            \u0026\u0026 in_array($action, [\u0027resize\u0027, \u0027forceResize\u0027, \u0027cropResize\u0027, \u0027cropZoom\u0027, \u0027zoomCrop\u0027, \u0027crop\u0027], true)) {\n+                            foreach ($args as $a) {\n+                                if (is_numeric($a) \u0026\u0026 (int) $a \u003e $max) {\n+                                    return false; // reject oversized derivative request\n+                                }\n+                            }\n+                        }\n+                        call_user_func_array([\u0026$medium, $action], $args);\n                     }\n                 }\n```\n\nDocument `system.images.max_dimension` (default 8000) so operators can tune it. A total-pixel ceiling (`width * height`) is a stricter alternative.",
  "id": "GHSA-4x9g-vw65-vvf9",
  "modified": "2026-08-14T19:23:38Z",
  "published": "2026-08-14T19:23:38Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://github.com/getgrav/grav/security/advisories/GHSA-4x9g-vw65-vvf9"
    },
    {
      "type": "ADVISORY",
      "url": "https://nvd.nist.gov/vuln/detail/CVE-2026-53653"
    },
    {
      "type": "WEB",
      "url": "https://github.com/getgrav/grav/commit/d9f9f0369a07ae5c96cde700c7949e1237b29cf6"
    },
    {
      "type": "WEB",
      "url": "https://github.com/getgrav/grav/commit/f4c0f42eea755cedad6f626b342c88d4cba72174"
    },
    {
      "type": "PACKAGE",
      "url": "https://github.com/getgrav/grav"
    },
    {
      "type": "WEB",
      "url": "https://github.com/getgrav/grav/releases/tag/1.7.53"
    },
    {
      "type": "WEB",
      "url": "https://github.com/getgrav/grav/releases/tag/2.0.0-rc.8"
    }
  ],
  "schema_version": "1.4.0",
  "severity": [
    {
      "score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:H/SC:N/SI:N/SA:N",
      "type": "CVSS_V4"
    }
  ],
  "summary": "Grav: Unauthenticated denial of service via unbounded image derivative dimensions"
}



Log in or create an account to share your comment.




Tags
Taxonomy of the tags.


Loading…

Loading…

Loading…

Forecast uses a logistic model when the trend is rising, or an exponential decay model when the trend is falling. Fitted via linearized least squares.

Sightings

Author Source Type Date Other

Nomenclature

  • Seen: The vulnerability was mentioned, discussed, or observed by the user.
  • Confirmed: The vulnerability has been validated from an analyst's perspective.
  • Published Proof of Concept: A public proof of concept is available for this vulnerability.
  • Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
  • Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
  • Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
  • Not confirmed: The user expressed doubt about the validity of the vulnerability.
  • Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.

Loading…

Detection rules are retrieved from Rulezet.

Loading…

Loading…

Loading…