Re: [PATCH] Fix hostaddr crash during non-blocking cancellation

2025-06-24 Thread Greg Sabino Mullane
01 looks sensible to me. I like 02 as well. Only quibble would be the name (tcp) as it doesn't really describe a class of things to be tested like the other things in PG_TEST_EXTRA. Something indicating a lack of socket? Just more verbose somehow? "tcp_only" perhaps? Cheers, Greg -- Crunchy Dat

[PATCH] Fix hostaddr crash during non-blocking cancellation

2025-05-05 Thread Jacob Champion
Hi all, A connection with only a hostaddr (no host) can't be cancelled via PQcreateCancel(), because we'll crash in emitHostIdentityInfo(). The problem is that the synthetic connhost entry we've created for cancellation has an incorrect type field, which causes the following code to make bad decis