RepoMirrors/haproxy
1
0
Fork 0
mirror of http://git.haproxy.org/git/haproxy.git/ synced 2025-01-11 16:29:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

[MINOR] halog: make SKIP_CHAR stop on field delimiters

Browse Source 
The SKIP_CHAR() macro did not consider field delimiters, causing the timer parser
to be able to search timers at wrong places when fed with TCP logs.
This commit is contained in:
Willy Tarreau 2011-07-10 17:25:33 +02:00
parent 812e7a73b2
commit c82570edec
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
contrib/halog/halog.c
View File

@ -36,7 +36,13 @@
#define MAXLINE 16384
#define QBITS 4
#define SKIP_CHAR(p,c) do { while (1) if (!*p) break; else if (*(p++) == c) break; } while (0)
const char sep[256] = {
[0] = 1,
[' '] = 1, ['\t'] = 1,
['\r'] = 1, ['\n'] = 1,
};
#define SKIP_CHAR(p,c) do { while (1) if (sep[(unsigned char)*p]) break; else if (*(p++) == c) break; } while (0)
/* [0] = err/date, [1] = req, [2] = conn, [3] = resp, [4] = data */
static struct eb_root timers[5] = {
@ -507,7 +513,7 @@ int main(int argc, char **argv)
p = b;
err = 0;
f = 0;
while (*p) {
while (!sep[(unsigned char)*p]) {
tps = str2ic(p);
if (tps < 0) {
tps = -1;
@ -580,7 +586,7 @@ int main(int argc, char **argv)
p = b;
err = 0;
f = 0;
while (*p) {
while (!sep[(unsigned char)*p]) {
array[f] = str2ic(p);
if (array[f] < 0) {
array[f] = -1;
@ -745,7 +751,7 @@ int main(int argc, char **argv)
p = b;
err = 0;
f = 0;
while (*p) {
while (!sep[(unsigned char)*p]) {
array[f] = str2ic(p);
if (array[f] < 0) {
array[f] = -1;
@ -813,7 +819,7 @@ int main(int argc, char **argv)
* parse the 5 timers to detect errors, it takes avg 55 ns per line.
*/
e = b; err = 0; f = 0;
while (*e) {
while (!sep[(unsigned char)*e]) {
array[f] = str2ic(e);
if (array[f] < 0) {
array[f] = -1;