compat/win32: correct for incorrect compiler warning
commitb4f52f09ae712e1a813375712b6e58be49255cd0
authorDerrick Stolee <derrickstolee@github.com>
Tue, 19 Jul 2022 18:45:54 +0000 (19 18:45 +0000)
committerJunio C Hamano <gitster@pobox.com>
Tue, 19 Jul 2022 19:51:34 +0000 (19 12:51 -0700)
treec3f277b4d616e44dd9c4a3d4543fa86faa1998e9
parentbbea4dcf42b28eb7ce64a6306cdde875ae5d09ca
compat/win32: correct for incorrect compiler warning

The 'win build' job of our CI build is failing with the following error:

compat/win32/syslog.c: In function 'syslog':
compat/win32/syslog.c:53:17: error: pointer 'pos' may be used after \
    'realloc' [-Werror=use-after-free]
   53 |                 memmove(pos + 2, pos + 1, strlen(pos));
    CC compat/poll/poll.o
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compat/win32/syslog.c:47:23: note: call to 'realloc' here
   47 |                 str = realloc(str, st_add(++str_len, 1));
      |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

However, between this realloc() and the use we have a line that resets
the value of 'pos'. Thus, this error is incorrect. It is likely due to a
new version of the compiler on the CI machines.

Instead of waiting for a new compiler, create a new variable to avoid
this error.

Signed-off-by: Derrick Stolee <derrickstolee@github.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
compat/win32/syslog.c