This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Don't assume __secure_getenv is available
- From: Jerry DeLisle <jvdelisle at charter dot net>
- To: Janne Blomqvist <blomqvist dot janne at gmail dot com>, Fortran List <fortran at gcc dot gnu dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 8 May 2017 10:33:50 -0700
- Subject: Re: [PATCH] Don't assume __secure_getenv is available
- Authentication-results: sourceware.org; auth=none
- References: <20170427125804.GZ1809@tucnak> <1493319051-25795-1-git-send-email-blomqvist.janne@gmail.com> <CAO9iq9GN3xyT0pDubNKmMUH8jg7t4aXzeHWkO2JANtBTUhvn7A@mail.gmail.com> <CAO9iq9FfUYEsTfYit0-SFYLXOk_6_2a7PL3YSLdD0TVbSVfb-g@mail.gmail.com>
On 05/07/2017 11:37 PM, Janne Blomqvist via fortran wrote:
> PING
>
> On Thu, Apr 27, 2017 at 9:55 PM, Janne Blomqvist
> <blomqvist.janne@gmail.com> wrote:
>> On Thu, Apr 27, 2017 at 9:50 PM, Janne Blomqvist
>> <blomqvist.janne@gmail.com> wrote:
>> [snip]
>>
>> And on top of that patch this simple typo fix:
>>
>> diff --git a/libgfortran/runtime/environ.c b/libgfortran/runtime/environ.c
>> index 969dcdf..f488e87 100644
>> --- a/libgfortran/runtime/environ.c
>> +++ b/libgfortran/runtime/environ.c
>> @@ -46,7 +46,7 @@ static char* weak_secure_getenv (const char*)
>> char *
>> secure_getenv (const char *name)
>> {
>> -#if SUPPORTS_WEAKREF && defined(HAVE__SECURE_GETENV)
>> +#if SUPPORTS_WEAKREF && defined(HAVE___SECURE_GETENV)
>> if (weak_secure_getenv)
>> return weak_secure_getenv (name);
>> #endif
>>
>>
>>
>> --
>> Janne Blomqvist
>
>
>
Looks Ok to me. I thought you might be waiting for Jakub to reply.
Thanks,
Jerry