diff options
author | dormando <dormando@rydia.net> | 2017-10-29 15:30:14 -0700 |
---|---|---|
committer | dormando <dormando@rydia.net> | 2017-10-29 15:30:14 -0700 |
commit | 69a4414178e1bd5f69db7810022a94799d7e1019 (patch) | |
tree | a1506e3849cb8bc203adbf11458f0dd1ecc03a4b /logger.c | |
parent | c22168bff7b198978525b3bf06d920517c9051a4 (diff) | |
download | memcached-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.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -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; } |