From: Tom Lane <tgl@sss.pgh.pa.us>
To: Richard Guo <guofenglinux@gmail.com>
cc: =?UTF-8?B?0JXQs9C+0YAg0KfQuNC90LTRj9GB0LrQuNC9?= <kyzevan23@mail.ru>,
        PostgreSQL mailing lists <pgsql-bugs@lists.postgresql.org>
Subject: Re: BUG #17583: Server crashes on executing CROSS JOIN with very big
 number of tables
In-reply-to: 
 <CAMbWs48JpVuP-89Pspros8JWy3EVMVPoVjF2-hnZHgatf=EKKw@mail.gmail.com>
References: <17583-33be55b9f981f75c@postgresql.org>
 <CAMbWs499ytQiH4mLMhRxRWP-iEUz3-DSinpAD-cUCtVo_23Wtg@mail.gmail.com>
 <CAMbWs48JpVuP-89Pspros8JWy3EVMVPoVjF2-hnZHgatf=EKKw@mail.gmail.com>
Comments: In-reply-to Richard Guo <guofenglinux@gmail.com>
	message dated "Fri, 12 Aug 2022 17:22:57 +0800"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <402039.1660418591.1@sss.pgh.pa.us>
Date: Sat, 13 Aug 2022 15:23:11 -0400
Message-ID: <402040.1660418591@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/402040.1660418591%40sss.pgh.pa.us>
Precedence: bulk

Richard Guo <guofenglinux@gmail.com> writes:
> On Fri, Aug 12, 2022 at 5:03 PM Richard Guo <guofenglinux@gmail.com> wrote:
>> Function transformFromClauseItem() is driven to stack overflow. We need
>> to add some check_stack_depth() checks here and maybe other more places.

> Attach a patch to add the check for transformFromClauseItem.

Pushed.  It's possible there are some other places, but I think most
of the parser doesn't need it because it'll call transformExpr which
already checks.

			regards, tom lane