Indeed, thanks for pointing it out. Its fixed now, in cvs and I also re-released the tarball (just for completeness).
Rainer > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:rsyslog- > [EMAIL PROTECTED] On Behalf Of Jakob Haufe > Sent: Tuesday, December 04, 2007 2:00 PM > To: [email protected] > Subject: Re: [rsyslog] rsyslog 1.20.0 preview available > > On Tue, 4 Dec 2007 11:23:26 +0100 > "Rainer Gerhards" <[EMAIL PROTECTED]> wrote: > > > Feedback is highly appreciated. If I updated the tarball, I'll > > announce that. Oh, yes, doc is not finished yet, so do not wonder... > > Just a small thing I already mentioned on the forum but unfortunately > got overlooked: > > ommysql, line 236: p+=sizeof(":ommysql:"); > > This adds increments the pointer one byte too far (trailing \0) and > thus eats the first byte of the hostname part. > > It's a bit unfortunate that strlen() doesn't count the \0 while > sizeof() does... > > Regards, > sur5r > > _______________________________________________ > rsyslog mailing list > http://lists.adiscon.net/mailman/listinfo/rsyslog _______________________________________________ rsyslog mailing list http://lists.adiscon.net/mailman/listinfo/rsyslog
