fix off-by-one error in IPAddress::longestCommonPrefix()

[folly.git] / folly / detail / IPAddress.h

diff --git a/folly/detail/IPAddress.h b/folly/detail/IPAddress.h
index 7f1095e05a93e4d38808932e0b698368bc3ec787..476f1fa10bf750fddf208e125a1383c88fd3c192 100644 (file)
--- a/folly/detail/IPAddress.h
+++ b/folly/detail/IPAddress.h
@@ -121,7 +121,7 @@ struct Bytes : private boost::noncopyable {
     // Compare a byte at a time. Note - I measured compared this with
     // going multiple bytes at a time (8, 4, 2 and 1). It turns out
     // to be 20 - 25% slower for 4 and 16 byte arrays.
-    while (byteIndex * 8 <= mask && one[byteIndex] == two[byteIndex]) {
+    while (byteIndex * 8 < mask && one[byteIndex] == two[byteIndex]) {
       ba[byteIndex] = one[byteIndex];
       ++byteIndex;
     }