| From: | Richard Guo <guofenglinux(at)gmail(dot)com> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
| Cc: | mahendrakar s <mahendrakarforpg(at)gmail(dot)com>, Егор Чиндяскин <kyzevan23(at)mail(dot)ru>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Stack overflow issue |
| Date: | 2022-08-24 11:54:36 |
| Message-ID: | CAMbWs48iswb9cXXnq1Ocy0WLdov__1eR=qenRP+a3x1PJiphJg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Aug 24, 2022 at 7:12 PM Richard Guo <guofenglinux(at)gmail(dot)com> wrote:
>
> On Wed, Aug 24, 2022 at 6:49 PM Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
> wrote:
>
>> On 2022-Aug-24, mahendrakar s wrote:
>>
>> > Hi,
>> > Can we have a parameter to control the recursion depth in these cases to
>> > avoid crashes?
>>
>> We already have one (max_stack_depth). The problem is lack of calling
>> the control function in a few places.
>
>
> Thanks Egor and Alexander for the work! I think we can just add
> check_stack_depth checks in these cases.
>
Attached adds the checks in these places. But I'm not sure about the
snowball case. Can we edit src/backend/snowball/libstemmer/*.c directly?
Thanks
Richard
| Attachment | Content-Type | Size |
|---|---|---|
| v1-0001-add-check_stack_depth-in-more-places.patch | application/octet-stream | 3.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2022-08-24 12:13:36 | Re: standby promotion can create unreadable WAL |
| Previous Message | Shinya Kato | 2022-08-24 11:50:08 | Re: Fix typo in func.sgml |