From eceb40be0f309a431abfd8898b9669cc1f5583c5 Mon Sep 17 00:00:00 2001
From: Niels Dossche <7771979+nielsdos@users.noreply.github.com>
Date: Wed, 17 Jan 2024 20:58:53 +0100
Subject: [PATCH] Field cleanups in xsl_object

---
 ext/xsl/php_xsl.h       | 7 +++----
 ext/xsl/xsltprocessor.c | 8 +++-----
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/ext/xsl/php_xsl.h b/ext/xsl/php_xsl.h
index 2d264b2976d16..b7a52dac5ac2f 100644
--- a/ext/xsl/php_xsl.h
+++ b/ext/xsl/php_xsl.h
@@ -49,14 +49,13 @@ extern zend_module_entry xsl_module_entry;
 #define XSL_SECPREF_CREATE_DIRECTORY 8
 #define XSL_SECPREF_READ_NETWORK 16
 #define XSL_SECPREF_WRITE_NETWORK 32
-/* Default == disable all write access ==  XSL_SECPREF_WRITE_NETWORK | XSL_SECPREF_CREATE_DIRECTORY | XSL_SECPREF_WRITE_FILE */
-#define XSL_SECPREF_DEFAULT 44
+/* Default == disable all write access */
+#define XSL_SECPREF_DEFAULT (XSL_SECPREF_WRITE_NETWORK | XSL_SECPREF_CREATE_DIRECTORY | XSL_SECPREF_WRITE_FILE)
 
 typedef struct _xsl_object {
 	void *ptr;
 	HashTable *parameter;
-	int hasKeys;
-	int securityPrefsSet;
+	bool hasKeys;
 	zend_long securityPrefs;
 	php_dom_xpath_callbacks xpath_callbacks;
 	php_libxml_node_object *doc;
diff --git a/ext/xsl/xsltprocessor.c b/ext/xsl/xsltprocessor.c
index ef53c6d24f050..7b22688c5d829 100644
--- a/ext/xsl/xsltprocessor.c
+++ b/ext/xsl/xsltprocessor.c
@@ -181,14 +181,14 @@ PHP_METHOD(XSLTProcessor, importStylesheet)
 		if (nodep && (nodep = nodep->children)) {
 			while (nodep) {
 				if (nodep->type == XML_ELEMENT_NODE && xmlStrEqual(nodep->name, (const xmlChar *) "key") && xmlStrEqual(nodep->ns->href, XSLT_NAMESPACE)) {
-					intern->hasKeys = 1;
+					intern->hasKeys = true;
 					break;
 				}
 				nodep = nodep->next;
 			}
 		}
 	} else {
-		intern->hasKeys = clone_docu;
+		intern->hasKeys = true;
 	}
 
 	if ((oldsheetp = (xsltStylesheetPtr)intern->ptr)) {
@@ -257,7 +257,7 @@ static xmlDocPtr php_xsl_apply_stylesheet(zval *id, xsl_object *intern, xsltStyl
 	intern->doc = emalloc(sizeof(php_libxml_node_object));
 	memset(intern->doc, 0, sizeof(php_libxml_node_object));
 
-	if (intern->hasKeys == 1) {
+	if (intern->hasKeys) {
 		doc = xmlCopyDoc(doc, 1);
 	} else {
 		object = Z_LIBXML_NODE_P(docp);
@@ -681,8 +681,6 @@ PHP_METHOD(XSLTProcessor, setSecurityPrefs)
 	intern = Z_XSL_P(id);
 	oldSecurityPrefs = intern->securityPrefs;
 	intern->securityPrefs = securityPrefs;
-	/* set this to 1 so that we know, it was set through this method. Can be removed, when we remove the ini setting */
-	intern->securityPrefsSet = 1;
 	RETURN_LONG(oldSecurityPrefs);
 }
 /* }}} end XSLTProcessor::setSecurityPrefs */