[Olsr-dev] status tip + BSD
Sven-Ola Tuecke
(spam-protected)
Thu Jan 29 08:41:08 CET 2009
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