Re: [PATCH] Fix out-of-bouds access (src/common/wchar.c) - Mailing list pgsql-hackers

From Ranier Vilela
Subject Re: [PATCH] Fix out-of-bouds access (src/common/wchar.c)
Date
Msg-id CAEudQAraZaRm_cN4d+34w+neT2aV-Cubmkc6T4TxXdg7QsKwUw@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Fix out-of-bouds access (src/common/wchar.c)  (Daniel Gustafsson <daniel@yesql.se>)
List pgsql-hackers
Em qui., 17 de fev. de 2022 às 10:18, Daniel Gustafsson <daniel@yesql.se> escreveu:
> On 17 Feb 2022, at 13:59, Ranier Vilela <ranier.vf@gmail.com> wrote:
>
> Em qui., 17 de fev. de 2022 às 09:52, Daniel Gustafsson <daniel@yesql.se <mailto:daniel@yesql.se>> escreveu:
> > On 17 Feb 2022, at 13:19, Ranier Vilela <ranier.vf@gmail.com <mailto:ranier.vf@gmail.com>> wrote:
>
> > 1. One #ifdef with a mistake, the correct is _WIN32 and not WIN32.
>
> Can you elaborate on this, we are using WIN32 pretty extensively in the code:
>
>  $ git grep "if[n]\{0,1\}def WIN32$"|wc -l
>      511
>  $ git grep "if[n]\{0,1\}def _WIN32$"|wc -l
>        2
>
> The _WIN32 cases are in the same ECPG testcase.
>
> Why would _WIN32 be correct in this case?
> Sorry, my fault.
>
> I only use _WIN32 and I jumped to conclusions very quickly.

Question remains though, should src/interfaces/ecpg/test/sql/sqlda.pgc really
be using WIN32 and not _WIN32, or doesn't it matter?  (or does it only matter
for consistency?) WIN32 and _WIN32 aren't very informative searchterms to use
for finding more information.
According the StackOverflow:
WIN32 -> SDK
_WIN32 -> Compiler (MSVC)

WIN32 with MingW (Windows) is 0?

For consistency, I think that will use only WIN32, if Postgres uses it extensively.

regards,

Ranier Vilela

pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: [PATCH] Fix out-of-bouds access (src/common/wchar.c)
Next
From: Ashutosh Bapat
Date:
Subject: Re: postgres_fdw and skip locked