https://bugzilla.balabit.com/show_bug.cgi?id=200 Gergely Nagy <algernon@balabit.hu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |algernon@balabit.hu --- Comment #2 from Gergely Nagy <algernon@balabit.hu> 2012-10-05 11:15:23 --- I tried to debug this, and it's definitely the LogProto refactoring that broke it, but I did not manage to find out yet why. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.

https://bugzilla.balabit.com/show_bug.cgi?id=200 Gergely Nagy <algernon@balabit.hu> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED --- Comment #3 from Gergely Nagy <algernon@balabit.hu> 2012-10-11 12:27:11 --- Right. This was a bit of a pain to figure out, as I had to understand Bazsi's recent logproto/logtransport rework, but the problem turned out to be really simple once understood: prior to the rework, the text transport was smart enough to recognise when it is in dgram-mode, and didn't expect a newline termination in that case. With the rework, the text transport was split, and a dgram transport was born. However, afunix was not modified, and it always set a text transport, even when operating on a unix-dgram. BUT /dev/log messages are not newline terminated, so they were accumulating in an internal buffer, waiting for the buffer to fill up. The obvious fix is to teach afunix to do the right thing, and set dgram transport as appropriate. I'll send a patch soon. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.

https://bugzilla.balabit.com/show_bug.cgi?id=200 Balazs Scheidler <bazsi@balabit.hu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bazsi@balabit.hu Resolution| |FIXED Status|ASSIGNED |RESOLVED --- Comment #5 from Balazs Scheidler <bazsi@balabit.hu> 2012-10-13 10:09:20 --- I couldn't find your branch first (my fault sorry), and by that time I also had a fix committed locally. I've slightly edited the commit message to reflect credits, but my patch also contained the testcase change, so we won't run into the same issue next time. commit 6022d4e1ad7f9c46c0b152f47c3f2d0383b7c27d Author: Balazs Scheidler <bazsi@balabit.hu> Date: Sat Oct 13 10:07:43 2012 +0200 unix-dgram: fixed message processing w/o terminating newlines In the last refactor, unix-dgram was switched to use the "text" LogProto Since unix-dgram clients don't always supply an NL character (such as on FreeBSD), this caused issues. This patch fixes the issue and also extends the testcases to cover this. Reported-by: Peter Czanik <czanik@balabit.hu> Signed-off-by: Gergely Nagy <algernon@balabit.hu> Signed-off-by: Balazs Scheidler <bazsi@balabit.hu> With this patch, this is considered fixed. Thanks. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.