This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v2] strace: Fix "over-optimization" flaw in strace.


On Apr 19 11:06, Daniel Santos wrote:
> Recent versions of gcc are optimizing away the TLS buffer allocated in
> main, so we need to tell gcc that it's really used.  RtlSecureZeroMemory
> accomplishes this while also inlining the memset.
> 
> Signed-off-by: Daniel Santos <daniel.santos@pobox.com>
> ---
>  winsup/utils/strace.cc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/winsup/utils/strace.cc b/winsup/utils/strace.cc
> index beab67b90..ae62cdc5f 100644
> --- a/winsup/utils/strace.cc
> +++ b/winsup/utils/strace.cc
> @@ -1191,7 +1191,7 @@ main (int argc, char **argv)
>       registry setting to 0x100000 (TOP_DOWN). */
>    char buf[CYGTLS_PADSIZE];
>  
> -  memset (buf, 0, sizeof (buf));
> +  RtlSecureZeroMemory (buf, sizeof (buf));
>    exit (main2 (argc, argv));
>  }
>  
> -- 
> 2.11.0

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: signature.asc
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]