Fix bogus affix-merging code.

tglsfdc · tglsfdc · commit f71d803c8de8 · 2018-04-12T18:39:51.000-04:00
NISortAffixes() compared successive compound affixes incorrectly, thus possibly failing to merge identical affixes, or (less likely) merging ones that shouldn't be merged. The user-visible effects of this are unclear, to me anyway. Per bug #15150 from Alexander Lakhin. It's been broken for a long time, so back-patch to all supported branches. Arthur Zakirov Discussion: https://postgr.es/m/152353327780.31225.13445405496721177988@wrigleys.postgresql.org
diff --git a/src/backend/tsearch/spell.c b/src/backend/tsearch/spell.c
@@ -1370,16 +1370,18 @@ NISortAffixes(IspellDict *Conf)
 		if ((Affix->flagflags & FF_COMPOUNDFLAG) && Affix->replen > 0 &&
 			isAffixInUse(Conf, (char) Affix->flag))
 		{
+			bool		issuffix = (Affix->type == FF_SUFFIX);
+
 			if (ptr == Conf->CompoundAffix ||
-				ptr->issuffix != (ptr - 1)->issuffix ||
+				issuffix != (ptr - 1)->issuffix ||
 				strbncmp((const unsigned char *) (ptr - 1)->affix,
 						 (const unsigned char *) Affix->repl,
 						 (ptr - 1)->len))
 			{
 				/* leave only unique and minimals suffixes */
 				ptr->affix = Affix->repl;
 				ptr->len = Affix->replen;
-				ptr->issuffix = (Affix->type == FF_SUFFIX) ? true : false;
+				ptr->issuffix = issuffix;
 				ptr++;
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -1370,16 +1370,18 @@ NISortAffixes(IspellDict *Conf)`
`1370`	`1370`	`if ((Affix->flagflags & FF_COMPOUNDFLAG) && Affix->replen > 0 &&`
`1371`	`1371`	`isAffixInUse(Conf, (char) Affix->flag))`
`1372`	`1372`	`{`
	`1373`	`+ bool issuffix = (Affix->type == FF_SUFFIX);`
	`1374`	`+`
`1373`	`1375`	`if (ptr == Conf->CompoundAffix \|\|`
`1374`		`- ptr->issuffix != (ptr - 1)->issuffix \|\|`
	`1376`	`+ issuffix != (ptr - 1)->issuffix \|\|`
`1375`	`1377`	`strbncmp((const unsigned char *) (ptr - 1)->affix,`
`1376`	`1378`	`(const unsigned char *) Affix->repl,`
`1377`	`1379`	`(ptr - 1)->len))`
`1378`	`1380`	`{`
`1379`	`1381`	`/* leave only unique and minimals suffixes */`
`1380`	`1382`	`ptr->affix = Affix->repl;`
`1381`	`1383`	`ptr->len = Affix->replen;`
`1382`		`- ptr->issuffix = (Affix->type == FF_SUFFIX) ? true : false;`
	`1384`	`+ ptr->issuffix = issuffix;`
`1383`	`1385`	`ptr++;`
`1384`	`1386`	`}`
`1385`	`1387`	`}`