mirror of
http://git.haproxy.org/git/haproxy.git/
synced 2024-12-19 01:54:37 +00:00
BUG/MINOR: log: LF upsets maxlen for UDP targets
A regression was introduced with 5464885
("MEDIUM: log/sink: re-work
and merge of build message API.").
For UDP targets, a final '\n' is systematically inserted, but with the
rework of the build message API, it is inserted after the maxlen
limitation has been enforced, so this can lead to the final message
becoming maxlen+1. For strict syslog servers that only accept up to
maxlen characters, this could be a problem.
To fix the regression, we take the final '\n' into account prior to
building the message, like it was done before the rework of the API.
This should be backported up to 2.4.
This commit is contained in:
parent
c103379847
commit
901f31bc9a
@ -1701,7 +1701,7 @@ static inline void __do_send_log(struct logsrv *logsrv, int nblogger, int level,
|
||||
}
|
||||
else {
|
||||
int i = 0;
|
||||
int totlen = logsrv->maxlen;
|
||||
int totlen = logsrv->maxlen - 1; /* save space for the final '\n' */
|
||||
|
||||
for (i = 0 ; i < nbelem ; i++ ) {
|
||||
iovec[i].iov_base = msg_header[i].ptr;
|
||||
|
Loading…
Reference in New Issue
Block a user