MIME-Version: 1.0
References: <Z64jD3u46gObCo1p@pryzbyj2023> <3718360.1739473209@sss.pgh.pa.us>
In-Reply-To: <3718360.1739473209@sss.pgh.pa.us>
From: Ranier Vilela <ranier.vf@gmail.com>
Date: Fri, 14 Feb 2025 09:17:09 -0300
Message-ID: <CAEudQApAqFQQ5F6uUfJ+EWn0OrRFwKpB_og2hcO4Dhh_YaeiCA@mail.gmail.com>
Subject: Re: pg17.3 PQescapeIdentifier() ignores len
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Justin Pryzby <pryzby@telsasoft.com>, pgsql-hackers@lists.postgresql.org
Content-Type: multipart/alternative; boundary="000000000000c87fc5062e192748"
Archived-At: <https://www.postgresql.org/message-id/CAEudQApAqFQQ5F6uUfJ%2BEWn0OrRFwKpB_og2hcO4Dhh_YaeiCA%40mail.gmail.com>
Precedence: bulk

--000000000000c87fc5062e192748
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Em qui., 13 de fev. de 2025 =C3=A0s 16:00, Tom Lane <tgl@sss.pgh.pa.us> esc=
reveu:

> Justin Pryzby <pryzby@telsasoft.com> writes:
> > The fprintf suggests that since 5dc1e42b4 PQescapeIdentifier ignores it=
s
> len.
>
> Ugh, yes.  Need something like the attached.
>
> 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.
>
Passes on standard tests at Windows 64 bits, msvc 2022 64 bits.

best regards,
Ranier Vilela

--000000000000c87fc5062e192748
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_quote gmail_quote_container"><div dir=
=3D"ltr" class=3D"gmail_attr">Em qui., 13 de fev. de 2025 =C3=A0s 16:00, To=
m Lane &lt;<a href=3D"mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>&gt; e=
screveu:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px=
 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Justin =
Pryzby &lt;<a href=3D"mailto:pryzby@telsasoft.com" target=3D"_blank">pryzby=
@telsasoft.com</a>&gt; writes:<br>
&gt; The fprintf suggests that since 5dc1e42b4 PQescapeIdentifier ignores i=
ts len.<br>
<br>
Ugh, yes.=C2=A0 Need something like the attached.<br>
<br>
FTR, 5dc1e42b4 et al were quite subtle patches done under extreme time<br>
pressure.=C2=A0 I wonder if they have any other issues.=C2=A0 More eyes on =
those<br>
patches would be welcome, now that they are public.<br></blockquote><div>Pa=
sses on standard tests at Windows 64 bits, msvc 2022 64 bits.</div><div><br=
></div><div>best regards,</div><div>Ranier Vilela</div></div></div>

--000000000000c87fc5062e192748--