public inbox for cygwin-cvs@sourceware.org help / color / mirror / Atom feed
From: Corinna Vinschen <corinna@sourceware.org> To: cygwin-cvs@sourceware.org Subject: [newlib-cygwin] Cygwin: fix initializing MEM_EXTENDED_PARAMETER, take 2 Date: Wed, 8 Sep 2021 08:17:10 +0000 (GMT) [thread overview] Message-ID: <20210908081710.38D093858407@sourceware.org> (raw) https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=8fba45f0ccc3b936edfb93671141bf3fe0b0ee5e commit 8fba45f0ccc3b936edfb93671141bf3fe0b0ee5e Author: Corinna Vinschen <corinna@vinschen.de> Date: Wed Sep 8 10:07:46 2021 +0200 Cygwin: fix initializing MEM_EXTENDED_PARAMETER, take 2 In https://cygwin.com/pipermail/cygwin/2021-September/249361.html Brian pointed out that initializing the structure would be more future proof, should the developers at Microsoft ever decide to split the Reserved field and use some bits of the struct for other purposes. Fixes: 3d322ac930ca ("Cygwin: fix initializing MEM_EXTENDED_PARAMETER") Signed-off-by: Corinna Vinschen <corinna@vinschen.de> Diff: --- winsup/cygwin/miscfuncs.cc | 6 ++---- winsup/cygwin/mmap.cc | 6 ++---- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/winsup/cygwin/miscfuncs.cc b/winsup/cygwin/miscfuncs.cc index b544e7125..024ad70ba 100644 --- a/winsup/cygwin/miscfuncs.cc +++ b/winsup/cygwin/miscfuncs.cc @@ -517,9 +517,8 @@ class thread_allocator THREAD_STACK_SLOT }; /* g++ 11.2 workaround: don't use initializer */ - MEM_EXTENDED_PARAMETER thread_ext; + MEM_EXTENDED_PARAMETER thread_ext = { 0 }; thread_ext.Type = MemExtendedParameterAddressRequirements; - thread_ext.Reserved = 0; thread_ext.Pointer = (PVOID) &thread_req; SIZE_T real_size = roundup2 (size, THREAD_STACK_SLOT); @@ -539,9 +538,8 @@ class thread_allocator THREAD_STACK_SLOT }; /* g++ 11.2 workaround: don't use initializer */ - MEM_EXTENDED_PARAMETER mmap_ext; + MEM_EXTENDED_PARAMETER mmap_ext = { 0 }; mmap_ext.Type = MemExtendedParameterAddressRequirements; - mmap_ext.Reserved = 0; mmap_ext.Pointer = (PVOID) &mmap_req; real_stackaddr = VirtualAlloc2 (GetCurrentProcess(), NULL, real_size, diff --git a/winsup/cygwin/mmap.cc b/winsup/cygwin/mmap.cc index 2e202a047..7982fd18e 100644 --- a/winsup/cygwin/mmap.cc +++ b/winsup/cygwin/mmap.cc @@ -207,9 +207,8 @@ MapView (HANDLE h, void *addr, size_t len, DWORD openflags, 0 }; /* g++ 11.2 workaround: don't use initializer */ - MEM_EXTENDED_PARAMETER mmap_ext; + MEM_EXTENDED_PARAMETER mmap_ext = { 0 }; mmap_ext.Type = MemExtendedParameterAddressRequirements; - mmap_ext.Reserved = 0; mmap_ext.Pointer = (PVOID) &mmap_req; alloc_type |= attached (prot) ? MEM_RESERVE : 0; @@ -1628,9 +1627,8 @@ fhandler_dev_zero::mmap (caddr_t *addr, size_t len, int prot, 0 }; /* g++ 11.2 workaround: don't use initializer */ - MEM_EXTENDED_PARAMETER mmap_ext; + MEM_EXTENDED_PARAMETER mmap_ext = { 0 }; mmap_ext.Type = MemExtendedParameterAddressRequirements; - mmap_ext.Reserved = 0; mmap_ext.Pointer = (PVOID) &mmap_req; base = VirtualAlloc2 (GetCurrentProcess(), *addr, len, alloc_type,
reply other threads:[~2021-09-08 8:17 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20210908081710.38D093858407@sourceware.org \ --to=corinna@sourceware.org \ --cc=cygwin-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).