[Olsr-dev] status tip + BSD

Hannes Gredler (spam-protected)
Thu Jan 29 09:13:54 CET 2009


you can use the "hg backout" command for undoing changes.

Sven-Ola Tuecke wrote:
> As a P.S: 
> 
> Please revert completely + immediately. That really looks ugly:
> 
> http://gredler.at/hg/olsrd/rev/c6a72905834a
> 
> As a hint: strncpy() does not what you obviously think about that function. 
> The function will _omit_ the trailing zero, e.g. strncpy(dst, src, 
> strlen(src)) generally leads to very long strings because the ASCIIZ zero is 
> not copied to the destination buffer. Also suspicous: 
> 
> strscpy(autobuf->buf+autobuf->len, s, len /* make openbsd shut up */)
> 
> Third parameter does not look like the strlen of s. strscpy is unknown under 
> linux etc.
> 
> // Sven-Ola
> 
> Am Donnerstag 29 Januar 2009 08:29:41 schrieb Sven-Ola Tuecke:
>> Hey aaron,
>>
>> that commit will break things. Please revise your strncpy() lines for +/- 1
>> errors, e.g. the following line does effectively nothing and should be
>> removed or revised:
>>
>> strncpy(buffer->buf, "", 0);
>>
>> // Sven-Ola
>>
>> Am Mittwoch 28 Januar 2009 21:41:20 schrieb L. Aaron Kaplan:
>>> hi!
>>>
>>> I tested tip on Mac OS X and OpenBSD.
>>> I commited some stuff against tip . Mainly making it compilable in
>>> OpenBSD.
>>>
>>> Please have a look at it in case you run VxWorks, FreeBSD, netBSD,
>>> openBSD or Mac OS X.
>>>
>>> I tested the patches in a small network with triangle with:
>>>
>>> 1x OpenBSD 4.1, 1x Mac OS X 10.5, 1 x Freifunk firmware 1.36
>>>
>>> works.
>>>
>>>
>>> a.
> 
> 
> 





More information about the Olsr-dev mailing list