[syslog-ng] problem receiving non-NULL terminated UDP messages
Balazs Scheidler
bazsi at balabit.hu
Sat Jul 30 22:50:06 CEST 2005
On Sat, 2005-07-30 at 14:22 +0200, e09f6a7593f8ae3994ea57e1117f67ec
wrote:
> Balazs Scheidler wrote:
> > Thanks for the detailed problem report. I have added a different
> > solution though, trying to merge the diverging code paths for SOCK_DGRAM
> > and SOCK_STREAM. I've committed a fix to my tla archive, the snapshot
> > for tomorrow should contain the fix, or you can apply the patch below to
> > your tree directly.
> >
> > Can you please test if this one works (I've compile tested and slightly
> > runtime tested and it seems to work fine).
>
> I have compiled syslog-ng-1.9.5+20050730.tar.gz and it does not seems to
> work and it cannot open udp source.
>
> Debugging with a single udp() source:
>
> afsocket_sd_init (s=0x8067f38, cfg=0x8062a20, persist=0x0) at afsocket.c:403
> self->connections = persist_config_fetch(persist,
> afsocket_sd_format_persist_name(self, FALSE));
> self->connections is null due persist==0x0
>
> afsocket.c:412
> if (!(self->flags & AFSOCKET_KEEP_ALIVE) || !self->connections ||
> !afsocket_open_s... bails out here
> self->flags = 0x701, self->connection=null
> self->flags & AFSOCKET_KEEP_ALIVE > 0, hmmm, udp keepalive?
>
> Note that persist if set to NULL at main.c:372 by calling cfg_init(cfg,
> NULL)
Sorry, did a stupid mistake, I tested a previous copy of the syslog-ng
binary and not the one which already contained the changes.
However I've extended the preliminary functional test program to also
cover tcp/udp message sending and added some more fixes.
Tomorrow's snapshot should contain the changes.
PS: let me know if a public Arch archive would be useful in addition to
the daily snapshots.
--
Bazsi
More information about the syslog-ng
mailing list