Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1phUGv-00031r-W1 for pgsql-bugs@arkaria.postgresql.org; Wed, 29 Mar 2023 11:47:21 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1phUGu-0004Xq-G5 for pgsql-bugs@arkaria.postgresql.org; Wed, 29 Mar 2023 11:47:20 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1phUGu-0004Xh-79 for pgsql-bugs@lists.postgresql.org; Wed, 29 Mar 2023 11:47:20 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1phUGr-0003bx-03 for pgsql-bugs@lists.postgresql.org; Wed, 29 Mar 2023 11:47:19 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 32TBlC7e3802071; Wed, 29 Mar 2023 07:47:12 -0400 From: Tom Lane To: Daniel Gustafsson cc: Alexander Lakhin , kyzevan23@mail.ru, pgsql-bugs@lists.postgresql.org Subject: Re: BUG #17874: Incorrect memory access at gistBuildCallback In-reply-to: References: <17874-63ca6c7ce42d2103@postgresql.org> <6341c61d-e6cd-bc68-8a8b-8b307b6f9359@gmail.com> Comments: In-reply-to Daniel Gustafsson message dated "Wed, 29 Mar 2023 12:11:09 +0200" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <3802069.1680090432.1@sss.pgh.pa.us> Content-Transfer-Encoding: quoted-printable Date: Wed, 29 Mar 2023 07:47:12 -0400 Message-ID: <3802070.1680090432@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Daniel Gustafsson writes: >> On 29 Mar 2023, at 12:00, Alexander Lakhin wrote: >> That issue was reported already as bug #16329: >> https://www.postgresql.org/message-id/flat/16329-7a6aa9b6fa1118a1%40pos= tgresql.org >> Maybe it's worth to register the fix proposed there for the next commit= fest... > Summarizing that thread and registering the proposed patch for the next > commitfest seems like the right thing to do. Yeah. We did fix the lack of test coverage that much of that thread was about, but it seems like we forgot the original bug (which I guess indicates that the coverage is still inadequate, since the valgrind buildfarm animals haven't noticed it...) regards, tom lane