summaryrefslogtreecommitdiff
path: root/logger.c
diff options
context:
space:
mode:
authordormando <dormando@rydia.net>2017-10-29 15:30:14 -0700
committerdormando <dormando@rydia.net>2017-10-29 15:30:14 -0700
commit69a4414178e1bd5f69db7810022a94799d7e1019 (patch)
treea1506e3849cb8bc203adbf11458f0dd1ecc03a4b /logger.c
parentc22168bff7b198978525b3bf06d920517c9051a4 (diff)
downloadmemcached-69a4414178e1bd5f69db7810022a94799d7e1019.tar.gz
make for loop more clear in logger watcher
This wasn't a bug, but add -1 here makes it more clear. The function will early exit in any cases where it would've overflowed here. thanks to shqking on github for the report
Diffstat (limited to 'logger.c')
-rw-r--r--logger.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/logger.c b/logger.c
index 25105ab..fdc2c19 100644
--- a/logger.c
+++ b/logger.c
@@ -721,7 +721,7 @@ enum logger_add_watcher_ret logger_add_watcher(void *c, const int sfd, uint16_t
return LOGGER_ADD_WATCHER_TOO_MANY;
}
- for (x = 0; x < WATCHER_LIMIT; x++) {
+ for (x = 0; x < WATCHER_LIMIT-1; x++) {
if (watchers[x] == NULL)
break;
}