Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitf976a77

Browse files
committed
Fix ts_headline() edge cases for empty query and empty search text.
tsquery's GETQUERY() macro is only safe to apply to a tsquerythat is known non-empty; otherwise it gives a pointer to garbage.Before commit5a617d7, ts_headline() avoided this pitfall, butonly in a very indirect, nonobvious way. (hlCover could not reachits TS_execute call, because if the query contains no lexemesthen hlFirstIndex would surely return -1.) After that commit,it fell into the trap, resulting in weird errors such as"unrecognized operator" and/or valgrind complaints. In HEAD,fix this by not calling TS_execute_locations() at all for anempty query. In the back branches, add a defensive check tohlCover() --- that's not fixing any live bug, but I judge thecode a bit too fragile as-is.Also, both mark_hl_fragments() and mark_hl_words() were carelessabout the possibility of empty search text: in the cases whereno match has been found, they'd end up telling mark_fragment() tomark from word indexes 0 to 0 inclusive, even when there is noword 0. This is harmless since we over-allocated the prs->wordsarray, but it does annoy valgrind. Fix so that the end index is -1and thus mark_fragment() will do nothing in such cases.Bottom line is that this fixes a live bug in HEAD, but in theback branches it's only getting rid of a valgrind nitpick.Back-patch anyway.Per report from Alexander Lakhin.Discussion:https://postgr.es/m/c27f642d-020b-01ff-ae61-086af287c4fd@gmail.com
1 parent2624de7 commitf976a77

File tree

3 files changed

+33
-2
lines changed

3 files changed

+33
-2
lines changed

‎src/backend/tsearch/wparser_def.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2038,6 +2038,9 @@ hlCover(HeadlineParsedText *prs, TSQuery query, int max_cover,
20382038
nextpmax;
20392039
hlCheckch;
20402040

2041+
if (query->size <=0)
2042+
return false;/* empty query matches nothing */
2043+
20412044
/*
20422045
* We look for the earliest, shortest substring of prs->words that
20432046
* satisfies the query. Both the pmin and pmax indices must be words
@@ -2342,7 +2345,8 @@ mark_hl_fragments(HeadlineParsedText *prs, TSQuery query, bool highlightall,
23422345
/* show the first min_words words if we have not marked anything */
23432346
if (num_f <=0)
23442347
{
2345-
startpos=endpos=curlen=0;
2348+
startpos=curlen=0;
2349+
endpos=-1;
23462350
for (i=0;i<prs->curwords&&curlen<min_words;i++)
23472351
{
23482352
if (!NONWORDTOKEN(prs->words[i].type))
@@ -2497,7 +2501,7 @@ mark_hl_words(HeadlineParsedText *prs, TSQuery query, bool highlightall,
24972501
if (bestlen<0)
24982502
{
24992503
curlen=0;
2500-
pose=0;
2504+
pose=-1;
25012505
for (i=0;i<prs->curwords&&curlen<min_words;i++)
25022506
{
25032507
if (!NONWORDTOKEN(prs->words[i].type))

‎src/test/regress/expected/tsearch.out

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2000,6 +2000,27 @@ to_tsquery('english','Lorem') && phraseto_tsquery('english','ullamcorper urna'),
20002000
<b>Lorem</b> ipsum <b>urna</b>. Nullam nullam <b>ullamcorper</b> <b>urna</b>
20012001
(1 row)
20022002

2003+
-- Edge cases with empty query
2004+
SELECT ts_headline('english',
2005+
'', ''::tsquery);
2006+
NOTICE: text-search query doesn't contain lexemes: ""
2007+
LINE 2: '', ''::tsquery);
2008+
^
2009+
ts_headline
2010+
-------------
2011+
2012+
(1 row)
2013+
2014+
SELECT ts_headline('english',
2015+
'foo bar', ''::tsquery);
2016+
NOTICE: text-search query doesn't contain lexemes: ""
2017+
LINE 2: 'foo bar', ''::tsquery);
2018+
^
2019+
ts_headline
2020+
-------------
2021+
foo bar
2022+
(1 row)
2023+
20032024
--Rewrite sub system
20042025
CREATE TABLE test_tsquery (txtkeyword TEXT, txtsample TEXT);
20052026
\set ECHO none

‎src/test/regress/sql/tsearch.sql

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -563,6 +563,12 @@ SELECT ts_headline('english',
563563
to_tsquery('english','Lorem') && phraseto_tsquery('english','ullamcorper urna'),
564564
'MaxFragments=100, MaxWords=100, MinWords=1');
565565

566+
-- Edge cases with empty query
567+
SELECT ts_headline('english',
568+
'',''::tsquery);
569+
SELECT ts_headline('english',
570+
'foo bar',''::tsquery);
571+
566572
--Rewrite sub system
567573

568574
CREATETABLEtest_tsquery (txtkeywordTEXT, txtsampleTEXT);

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp