Please, apply patch of tsearch for current CVS & 7.3.1

From: Teodor Sigaev <teodor(at)stack(dot)net>
To: "Magnus Naeslund(f)" <mag(at)fbab(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Subject: Please, apply patch of tsearch for current CVS & 7.3.1
Date: 2002-12-04 08:32:54
Message-ID: 3DEDBDB6.1060508@stack.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Thank you very much, you catch it :). This bug had a long life, because it
exists if and only if locale of postmaster
was a different from C (or ru_RU.KOI8-R).

Please, apply patch for current CVS & 7.3.1

Magnus Naeslund(f) wrote:
> Ok, I nailed the bug, but i'm not sure what the correct fix is.
> Attached tsearch_morph.diff that remedies this problem by avoiding it.
> Also there's a debug aid patch if someone would like to know how i
> finally found it out :)
>
> There problem in the lemmatize() function is that GETDICT(...) returned
> a value not handled (BYLOCALE).
> The value (-1) and later used as an index into the dicts[] array.
> After that everything went berserk stack went crazy somehow so trapping
> the fault sent me to the wrong place, and every time i read the value it
> was positive ;)
>
> So now i just return the initial word passed to the lemmatize function,
> because i don't know what to do with it.
>
> So you tsearch guys will have to work it out :)
>

--
Teodor Sigaev
teodor(at)stack(dot)net

Attachment Content-Type Size
tsearch_patch.gz application/gzip 251 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Frank van Vugt 2002-12-04 09:01:23 Re: Segmentation fault in 7.3
Previous Message Dave Page 2002-12-04 08:27:26 Re: [GENERAL] PostgreSQL Global Development Group Announces