Pending patches for INN?

Julien ÉLIE julien at trigofacile.com
Sat May 23 13:14:53 UTC 2015


Hi Richard,

>> Regarding your first patch for innxmit, shouldn't
>>    static char *buff;
>> be changed to
>>    static char *buff = NULL;
>> as you no longer use xmalloc, but only xrealloc?
>
> Objects with static storage duration are initialized automatically, in
> this case to a null pointer (C99 6.7.8#10) - so the two lines are
> equivalent.

OK, thanks for the precision.



>> As for actsync.c, maybe we could add the following test at the beginning
>> of get_active()?
>>
>> @@ -742,7 +742,9 @@
>>       if (len == NULL)
>>           die("internal error #1: len is NULL");
>>       if (errs == NULL)
>> -        die("internal error #2: errs in NULL");
>> +        die("internal error #2: errs is NULL");
>> +    if (host == NULL)
>> +        die("internal error #10: host is NULL");
>>       if (D_BUG)
>>           warn("STATUS: obtaining active file from %s", host);
>
> Yes, that wouldn't hurt.

Added, then.
Thanks again.

-- 
Julien ÉLIE

« Oublie les injures, n'oublie jamais les bienfaits. »


More information about the inn-workers mailing list