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