153aa9179Sopenharmony_ciFrom e301865e69b9b834f7b777dc58a9cee40ae056b2 Mon Sep 17 00:00:00 2001
253aa9179Sopenharmony_ciFrom: Nick Wellnhofer <wellnhofer@aevum.de>
353aa9179Sopenharmony_ciDate: Thu, 9 Mar 2023 05:34:38 +0100
453aa9179Sopenharmony_ciSubject: [PATCH] regexp: Fix checks for eliminated transitions
553aa9179Sopenharmony_ci
653aa9179Sopenharmony_ci'to' can be set to -1 or -2 when eliminating transitions, so check for
753aa9179Sopenharmony_ciall negative values.
853aa9179Sopenharmony_ci
953aa9179Sopenharmony_ciReference:https://github.com/GNOME/libxml2/commit/e301865e69b9b834f7b777dc58a9cee40ae056b2
1053aa9179Sopenharmony_ciConflict:NA
1153aa9179Sopenharmony_ci
1253aa9179Sopenharmony_ci---
1353aa9179Sopenharmony_ci xmlregexp.c | 12 ++++++------
1453aa9179Sopenharmony_ci 1 file changed, 6 insertions(+), 6 deletions(-)
1553aa9179Sopenharmony_ci
1653aa9179Sopenharmony_cidiff --git a/xmlregexp.c b/xmlregexp.c
1753aa9179Sopenharmony_ciindex 24f9fc0..df0626c 100644
1853aa9179Sopenharmony_ci--- a/xmlregexp.c
1953aa9179Sopenharmony_ci+++ b/xmlregexp.c
2053aa9179Sopenharmony_ci@@ -607,7 +607,7 @@ xmlRegEpxFromParse(xmlRegParserCtxtPtr ctxt) {
2153aa9179Sopenharmony_ci 
2253aa9179Sopenharmony_ci 	    for (j = 0;j < state->nbTrans;j++) {
2353aa9179Sopenharmony_ci 		trans = &(state->trans[j]);
2453aa9179Sopenharmony_ci-		if ((trans->to == -1) || (trans->atom == NULL))
2553aa9179Sopenharmony_ci+		if ((trans->to < 0) || (trans->atom == NULL))
2653aa9179Sopenharmony_ci 		    continue;
2753aa9179Sopenharmony_ci                 atomno = stringRemap[trans->atom->no];
2853aa9179Sopenharmony_ci 		if ((trans->atom->data != NULL) && (transdata == NULL)) {
2953aa9179Sopenharmony_ci@@ -2783,11 +2783,11 @@ xmlFAComputesDeterminism(xmlRegParserCtxtPtr ctxt) {
3053aa9179Sopenharmony_ci 		/* t1->nd = 1; */
3153aa9179Sopenharmony_ci 		continue;
3253aa9179Sopenharmony_ci 	    }
3353aa9179Sopenharmony_ci-	    if (t1->to == -1) /* eliminated */
3453aa9179Sopenharmony_ci+	    if (t1->to < 0) /* eliminated */
3553aa9179Sopenharmony_ci 		continue;
3653aa9179Sopenharmony_ci 	    for (i = 0;i < transnr;i++) {
3753aa9179Sopenharmony_ci 		t2 = &(state->trans[i]);
3853aa9179Sopenharmony_ci-		if (t2->to == -1) /* eliminated */
3953aa9179Sopenharmony_ci+		if (t2->to < 0) /* eliminated */
4053aa9179Sopenharmony_ci 		    continue;
4153aa9179Sopenharmony_ci 		if (t2->atom != NULL) {
4253aa9179Sopenharmony_ci 		    if (t1->to == t2->to) {
4353aa9179Sopenharmony_ci@@ -2825,11 +2825,11 @@ xmlFAComputesDeterminism(xmlRegParserCtxtPtr ctxt) {
4453aa9179Sopenharmony_ci 	    if (t1->atom == NULL) {
4553aa9179Sopenharmony_ci 		continue;
4653aa9179Sopenharmony_ci 	    }
4753aa9179Sopenharmony_ci-	    if (t1->to == -1) /* eliminated */
4853aa9179Sopenharmony_ci+	    if (t1->to < 0) /* eliminated */
4953aa9179Sopenharmony_ci 		continue;
5053aa9179Sopenharmony_ci 	    for (i = 0;i < transnr;i++) {
5153aa9179Sopenharmony_ci 		t2 = &(state->trans[i]);
5253aa9179Sopenharmony_ci-		if (t2->to == -1) /* eliminated */
5353aa9179Sopenharmony_ci+		if (t2->to < 0) /* eliminated */
5453aa9179Sopenharmony_ci 		    continue;
5553aa9179Sopenharmony_ci 		if (t2->atom != NULL) {
5653aa9179Sopenharmony_ci                     /*
5753aa9179Sopenharmony_ci@@ -2843,7 +2843,7 @@ xmlFAComputesDeterminism(xmlRegParserCtxtPtr ctxt) {
5853aa9179Sopenharmony_ci 			t2->nd = 1;
5953aa9179Sopenharmony_ci 			last = t1;
6053aa9179Sopenharmony_ci 		    }
6153aa9179Sopenharmony_ci-		} else if (t1->to != -1) {
6253aa9179Sopenharmony_ci+		} else {
6353aa9179Sopenharmony_ci 		    /*
6453aa9179Sopenharmony_ci 		     * do the closure in case of remaining specific
6553aa9179Sopenharmony_ci 		     * epsilon transitions like choices or all
6653aa9179Sopenharmony_ci-- 
6753aa9179Sopenharmony_ci2.27.0
6853aa9179Sopenharmony_ci
69