INN patches following static analysis

Julien ÉLIE julien at trigofacile.com
Sun May 3 17:18:44 UTC 2015


Hi Richard,

>>> [PATCH 4/5] Paranoid checking of AUTHINFO GENERIC reply
>>>
>>> + Reply("%d Program error occurred\r\n", NNTP_FAIL_ACTION,
>>> +       strerror(errno));
>>
>> I removed the errno part in the commit.
>
> Oh, well spotted, sorry about that.

No problem.  I wish to thank you again for your help to improve the 
robustness of INN.



>> Subject: [PATCH 3/5] Correct sense of PERMgeneric reply
>>
>> The return value has been wrong for ~15 years.
>
> I bet that nobody ever used AUTHINFO GENERIC with INN 2.x...

In fact, since INN 2.4.0 because the code up to 2.3.5 seems right.
If you don't mind, I will add your name to our CONTRIBUTORS file as I 
see you currently are not mentioned herein.

-- 
Julien ÉLIE

« – Vous refusez de porter pilum ?
   – Ben… On préfère se faire porter pâles… » (Astérix)


More information about the inn-workers mailing list