| From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Fix a strict aliasing violation |
| Date: | 2025-12-01 15:46:48 |
| Message-ID: | E1vQ66x-002IXC-2u@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Fix a strict aliasing violation
This one is almost a textbook example of an aliasing violation, and it
is straightforward to fix, so clean it up. (The warning only shows up
if you remove the -fno-strict-aliasing option.) Also, move the code
after the error checking. Doesn't make a difference technically, but
it seems strange to do actions before errors are checked.
Reported-by: Tatsuo Ishii <ishii(at)postgresql(dot)org>
Reviewed-by: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Discussion: https://www.postgresql.org/message-id/flat/20240724.155525.366150353176322967.ishii%40postgresql.org
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/2fcc5a715130fbe9fb6eadf338e3bfe560eb0cb5
Modified Files
--------------
src/backend/replication/logical/origin.c | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Álvaro Herrera | 2025-12-01 16:34:47 | pgsql: Fix ON CONFLICT ON CONSTRAINT during REINDEX CONCURRENTLY |
| Previous Message | Michael Paquier | 2025-12-01 07:22:34 | pgsql: Move WAL sequence code into its own file |