Hi, On 2025-02-13 14:00:09 -0500, Tom Lane wrote: > Justin Pryzby <pry...@telsasoft.com> writes: > > The fprintf suggests that since 5dc1e42b4 PQescapeIdentifier ignores its > > len. > > Ugh, yes. Need something like the attached.
I just pushed this fix, together with an expansion of test_escape.c. With the expanded test both uses of strlen() are detected. > FTR, 5dc1e42b4 et al were quite subtle patches done under extreme time > pressure. I wonder if they have any other issues. More eyes on those > patches would be welcome, now that they are public. Indeed. Greetings, Andres Freund