| From: | "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com> |
|---|---|
| To: | 'Jan Nidzwetzki' <jan(at)planetscale(dot)com>, 'Ayush Tiwari' <ayushtiwari(dot)slg01(at)gmail(dot)com>, ♂π≌26218 <1991230470(at)qq(dot)com> |
| Cc: | pgsql-bugs <pgsql-bugs(at)lists(dot)postgresql(dot)org>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
| Subject: | RE: START_REPLICATION silently truncates an overlong LSN component |
| Date: | 2026-10-20 07:36:00 |
| Message-ID: | OS7PR01MB18317EED85B1A72C69B226630F5672@OS7PR01MB18317.jpnprd01.prod.outlook.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Hi Jan,
> While looking for other places that need the same change, I found that
> readTimeLineHistory() in src/backend/access/transam/timeline.c still
> parses the switchpoint with sscanf("%X/%08X"). f31d6fbc31d fixed the
> frontend copy of this code in pg_rewind, but not the backend one. The
> attached 0003 patch converts it the same way and adds a TAP test to
> 004_timeline_switch.pl, including a case for the trailing characters
> that are now rejected. As with the test in 0001, I'm not sure this one
> needs to be committed.
I also checked the source, and found that read_backup_label() parses the %X/%08X
in fscanf(). Can it be also replaced?
Best regards,
Hayato Kuroda
FUJITSU LIMITED
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2026-10-20 07:41:49 | Re: BUG #19760: CLUSTER loses a row the transaction inserted and then updated in a rolled-back subtransaction |
| Previous Message | Michael Paquier | 2026-10-20 07:12:59 | Re: BUG #19763: ManualBug2 NULL pointer dereference in make_build_data via examine_expression bypass of lookup_var_a |