Fixed bug with incorrectly checked 4xx status codes when --4xx-to-unique-count flag is specified.

mikhailv · allinurl · commit f08daae8fd2c · 2025-03-11T11:29:17.000-05:00
The bug was introduced in commit 6d2cb7e.
diff --git a/src/gstorage.c b/src/gstorage.c
@@ -1388,7 +1388,7 @@ static int
 include_uniq (GLogItem *logitem) {
   int u = conf.client_err_to_unique_count;
 
-  if (!logitem->status || (logitem->status / 100) != 4 || (u && (logitem->status / 100) == '4'))
+  if (!logitem->status || (logitem->status / 100) != 4 || (u && (logitem->status / 100) == 4))
     return 1;
   return 0;
 }

Original file line number	Diff line number	Diff line change
`@@ -1388,7 +1388,7 @@ static int`
`1388`	`1388`	`include_uniq (GLogItem *logitem) {`
`1389`	`1389`	`int u = conf.client_err_to_unique_count;`
`1390`	`1390`
`1391`		`- if (!logitem->status \|\| (logitem->status / 100) != 4 \|\| (u && (logitem->status / 100) == '4'))`
	`1391`	`+ if (!logitem->status \|\| (logitem->status / 100) != 4 \|\| (u && (logitem->status / 100) == 4))`
`1392`	`1392`	`return 1;`
`1393`	`1393`	`return 0;`
`1394`	`1394`	`}`