From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | p(dot)nekrasov(at)fobos-nt(dot)ru, pgsql-bugs(at)lists(dot)postgresql(dot)org |
Subject: | Re: BUG #18614: [ECPG] out of bound in DecodeDateTime |
Date: | 2024-10-22 05:53:50 |
Message-ID: | Zxc97vmDiFzNYylT@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
On Fri, Oct 18, 2024 at 02:51:32PM +0900, Michael Paquier wrote:
> It is not entirely the fault of this patch, because the same error is
> done a couple of lines above when using an incorrect hour number with
> timestamp_to_asc3. The correct thing to do would be in the lines of
> what num_test2.pgc does with its check_errno(), I think where we'd
> check for the errno returned by PGTYPEStimestamp_from_asc() and
> printf() its information in the output of the test. And we should do
> that for both the old timestamp_to_asc3 and your new timestamp_to_asc4
> if we want to be completely correct.
And here is what I had in mind, checking the error code on top of the
output. The output is really optional, but that maps with 0 with the
offset applied, so I've just kept it.
--
Michael
Attachment | Content-Type | Size |
---|---|---|
0001-ecpg-Fix-out-of-bound-read-in-DecodeDateTime.patch | text/x-diff | 12.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2024-10-22 07:33:21 | Re: BUG #18568: BUG: Result wrong when do group by on partition table! |
Previous Message | Tender Wang | 2024-10-22 01:38:49 | Re: BUG #18664: Assert in BeginCopyTo() fails when source DML query rewritten with notifying rule |