[HACKERS] Empty dictionary file when creating text search dictionary

2009-12-01 Thread Robert Gravsjö

Found this a couple of weeks back and just re-tested against head:

CREATE TEXT SEARCH DICTIONARY with an empty thesaurus file will crasch 
the backend.


To reproduce:
$ echo   $(pg_config --sharedir)/tsearch_data/thesaurus_empty.ths

Then use this thesaurus to create a text search dictionary:
CREATE TEXT SEARCH DICTIONARY thesaurus_astro (
TEMPLATE = thesaurus,
DictFile = thesaurus_empty,
Dictionary = english_stem
);

It doesn't matter if the file is compleatly empty or just have comments. 
The result is the same.

Add a word:
$ echo foo: bar  $(pg_config 
--sharedir)/tsearch_data/thesaurus_empty.ths


Creating the dictionary will now work.

Sorry  I have no patch to attach.

Regards,
roppert


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


[HACKERS] Empty dictionary file when creating text search dictionary

2009-11-30 Thread Robert Gravsjö

Found this a couple of weeks back and just re-tested against head:

CREATE TEXT SEARCH DICTIONARY with an empty thesaurus file will crasch 
the backend.


To reproduce:
$ echo   $(pg_config --sharedir)/tsearch_data/thesaurus_empty.ths

Then use this thesaurus to create a text search dictionary:
CREATE TEXT SEARCH DICTIONARY thesaurus_astro (
TEMPLATE = thesaurus,
DictFile = thesaurus_empty,
Dictionary = english_stem
);

It doesn't matter if the file is compleatly empty or just have comments. 
The result is the same.

Add a word:
$ echo foo: bar  $(pg_config 
--sharedir)/tsearch_data/thesaurus_empty.ths


Creating the dictionary will now work.

Sorry  I have no patch to attach.

Regards,
roppert

PS. I happend to send this mail from a wrong, non-subscribed, address 
earlier so if it turns up duplicated I apologize.


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers