Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAL3m8eCM+r_=t+V_137JqphV3HVWz3Lj=6hqZw6jOZVe6dTZzg@mail.gmail.com>
Date: Fri, 28 Dec 2012 18:32:00 -0500
From: Strake <strake888@...il.com>
To: musl@...ts.openwall.com
Subject: Re: [PATCH] include/stdio.h: declare *asprintf with _BSD_SOURCE

On 28/12/2012, Rich Felker <dalias@...ifal.cx> wrote:
> On Fri, Dec 28, 2012 at 09:23:17AM -0500, Strake wrote:
>> *asprintf is indeed declared in BSD stdio.h, so do the same in musl.
>
> This change should be fine.
>
>> commit 389e354a13ee040a9a413e61639be6d68ff1bc03
>> Author: strake <strake888@...il.com>
>> Date:   Fri Dec 28 09:20:13 2012 -0500
>>
>>     include/stdio.h: declare *asprintf with _BSD_SOURCE
>>
>> diff --git a/include/stdio.h b/include/stdio.h
>> index 6e8e645..79b2c0f 100644
>> --- a/include/stdio.h
>> +++ b/include/stdio.h
>> @@ -177,9 +177,12 @@ int putw(int, FILE *);
>>  char *fgetln(FILE *, size_t *);
>>  #endif
>>
>> -#ifdef _GNU_SOURCE
>> +#if defined _GNU_SOURCE || defined _BSD_SOURCE
>>  int asprintf(char **, const char *, ...);
>>  int vasprintf(char **, const char *, va_list);
>> +#endif
>> +
>> +#ifdef _GNU_SOURCE
>
> There's already an #if block just above that's GNU-or-BSD. I'm just
> moving these two declarations into it rather than making a new one.
>
> Rich
>

Thanks.

Cheers,
Strake

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.