Re: BUG #15150: Reading uninitialised value in NISortAffixes (tsearch/spell.c)

From: Arthur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: exclusion(at)gmail(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #15150: Reading uninitialised value in NISortAffixes (tsearch/spell.c)
Date: 2018-04-13 19:05:30
Message-ID: 20180413190529.GA6210@arthur.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Fri, Apr 13, 2018 at 01:51:00PM -0400, Tom Lane wrote:
> > I attached the patch. It fixes the following:
> > - show an error if actual number of affix aliases is greater than
> > initial number. I wonder is it necessary. But I think it is better to
> > raise an error than crash, if you set wrong number for AF flag.
>
> Good idea, but I tweaked the message wording a bit.
>
> > The code coverage still isn't 100% for spell.c. But it is better than
> > earlier.
>
> Indeed. Pushed, thanks!

Thank you!

--
Arthur Zakirov
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message PG Bug reporting form 2018-04-14 10:39:21 BUG #15155: table_to_xmlschema() ignores string restriction when generating XSD
Previous Message Tom Lane 2018-04-13 17:51:00 Re: BUG #15150: Reading uninitialised value in NISortAffixes (tsearch/spell.c)