Fix possible use of uninitialised value in ts_headline()

feodor · feodor · commit bf73016e01f0 · 2016-04-08T21:25:32.000+03:00
Found during investigation of failure of skink buildfarm member and its
valgrind report.

Backpatch to all supported branches
diff --git a/src/backend/tsearch/wparser_def.c b/src/backend/tsearch/wparser_def.c
@@ -2441,6 +2441,8 @@ mark_hl_words(HeadlineParsedText *prs, TSQuery query, int highlight,
 			}
 			else
 			{					/* shorter cover :((( */
+				if (i > q)
+					i = q;
 				for (; curlen > min_words; i--)
 				{
 					if (!NONWORDTOKEN(prs->words[i].type))

Original file line number	Diff line number	Diff line change
`@@ -2441,6 +2441,8 @@ mark_hl_words(HeadlineParsedText *prs, TSQuery query, int highlight,`
`2441`	`2441`	`}`
`2442`	`2442`	`else`
`2443`	`2443`	`{ /* shorter cover :((( */`
	`2444`	`+ if (i > q)`
	`2445`	`+ i = q;`
`2444`	`2446`	`for (; curlen > min_words; i--)`
`2445`	`2447`	`{`
`2446`	`2448`	`if (!NONWORDTOKEN(prs->words[i].type))`