Page MenuHomePhabricator
Authored By
SomeRandomDeveloper
Mar 19 2026, 8:09 PM
Size
885 B
Referenced Files
None
Subscribers
None

T419168.patch

From 42ca7771772cb73b57af8c9661b003f6e483334f Mon Sep 17 00:00:00 2001
From: SomeRandomDeveloper <git@srd.dev>
Date: Thu, 19 Mar 2026 21:08:52 +0100
Subject: [PATCH] SECURITY: Always call engine destructor to fix memory leak
Bug: T419168
Change-Id: I68ffa6f1ff290f6d04c2bbc9d3f1d95edec64a27
---
includes/EngineFactory.php | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/includes/EngineFactory.php b/includes/EngineFactory.php
index 14c4dfa..45ea23e 100644
--- a/includes/EngineFactory.php
+++ b/includes/EngineFactory.php
@@ -125,6 +125,11 @@ class EngineFactory {
}
public function destroyEngineForParser( Parser $parser ): void {
+ if ( isset( $this->engineForParser[$parser] ) ) {
+ // T419168: Call destroy() manually to prevent memory leaks
+ $this->engineForParser[$parser]->destroy();
+ }
+
unset( $this->engineForParser[$parser] );
}
}
--
2.53.0

File Metadata

Mime Type
text/x-diff
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
23657074
Default Alt Text
T419168.patch (885 B)

Event Timeline