[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: canon-host errors
From: |
Derek Price |
Subject: |
Re: canon-host errors |
Date: |
Tue, 13 Sep 2005 09:53:36 -0400 |
User-agent: |
Mozilla Thunderbird 1.0.6 (Windows/20050716) |
Jim Meyering wrote:
>Derek Price <address@hidden> wrote:
>
>
>>I've installed the attached patch. It is almost identical to my
>>previous one, with a few extra portability and typo fixes.
>>
>>2005-09-12 Derek Price <address@hidden>
>>
>> * modules/canon-host: Add canon-host.h. Depend on getaddrinfo. Make
>> LGPL.
>> * modules/getaddrinfo: Add link to opengroup spec. Depend on strdup.
>>
>>
>
>Thanks for doing all of that.
>
>There is one small problem.
>When strdup returns NULL, canon_host_r
>returns NULL without setting *cherror.
>Shouldn't that be fixed?
>
>
Yes. Thanks. I've installed the attached patch.
2005-09-13 Derek Price <address@hidden>
* canon-host.c (canon_host_r): Set *cherror on memory allocation
failure.
Reported by Jim Meyering <address@hidden>.
>I've checked in these cosmetic changes:
>
>
Thanks.
>That file was using an indentation style different from the style
>used in nearly every other source file. I ran it through GNU indent.
>
>
Thanks. I meant to do that and forgot.
>Finally, we prefer to use an active voice (rather than passive voice)
>in comments. So I changed this:
>
>
Thanks.
Regards,
Derek
--
Derek R. Price
CVS Solutions Architect
Ximbiot <http://ximbiot.com>
v: +1 717.579.6168
f: +1 717.234.3125
<mailto:address@hidden>
Index: lib/canon-host.c
===================================================================
RCS file: /cvsroot/gnulib/gnulib/lib/canon-host.c,v
retrieving revision 1.20
diff -u -p -r1.20 canon-host.c
--- lib/canon-host.c 13 Sep 2005 12:37:48 -0000 1.20
+++ lib/canon-host.c 13 Sep 2005 12:59:11 -0000
@@ -72,6 +72,8 @@ canon_host_r (char const *host, int *che
if (!status)
{
retval = strdup (res->ai_canonname);
+ if (!retval && cherror)
+ *cherror = EAI_MEMORY;
freeaddrinfo (res);
}
else if (cherror)
- Re: canon-host errors, (continued)
- Re: AC 2.59 incompatibility in getaddrinfo.h, Simon Josefsson, 2005/09/08
- Re: canon-host errors, Jim Meyering, 2005/09/04
- Re: canon-host errors, Derek Price, 2005/09/05
- Re: canon-host errors, Simon Josefsson, 2005/09/06
- Re: canon-host errors, Derek Price, 2005/09/13
- Re: canon-host errors, Jim Meyering, 2005/09/13
- Re: canon-host errors,
Derek Price <=