From 5880a9a6bd97c0f9ac8fc4f30110fe023f484746 Mon Sep 17 00:00:00 2001 From: Nick Wellnhofer Date: Tue, 10 Dec 2024 16:52:05 +0100 Subject: [PATCH] [CVE-2024-56171] Fix use-after-free after xmlSchemaItemListAdd xmlSchemaItemListAdd can reallocate the items array. Update local variables after adding item in - xmlSchemaIDCFillNodeTables - xmlSchemaBubbleIDCNodeTables Fixes #828. --- xmlschemas.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/xmlschemas.c b/xmlschemas.c index 1b3c524f2..95be97c96 100644 --- a/xmlschemas.c +++ b/xmlschemas.c @@ -23374,6 +23374,7 @@ xmlSchemaIDCFillNodeTables(xmlSchemaValidCtxtPtr vctxt, } if (xmlSchemaItemListAdd(bind->dupls, bind->nodeTable[j]) == -1) goto internal_error; + dupls = (xmlSchemaPSVIIDCNodePtr *) bind->dupls->items; /* * Remove the duplicate entry from the IDC node-table. */ @@ -23590,6 +23591,8 @@ xmlSchemaBubbleIDCNodeTables(xmlSchemaValidCtxtPtr vctxt) goto internal_error; } xmlSchemaItemListAdd(parBind->dupls, parNode); + dupls = (xmlSchemaPSVIIDCNodePtr *) + parBind->dupls->items; } else { /* * Add the node-table entry (node and key-sequence) of -- GitLab