Fix error in Blob::operator== 68/70168/2
authorDave Rigby <daver@couchbase.com>
Mon, 21 Nov 2016 17:22:17 +0000 (17:22 +0000)
committerDave Rigby <daver@couchbase.com>
Tue, 22 Nov 2016 10:58:42 +0000 (10:58 +0000)
Fix typo in Blob::operaror== - incorrect comparision of
extMetaLen. Introduced by http://review.couchbase.org/69477

Bug identified by Coverity:

    CID 153111:  Incorrect expression  (CONSTANT_EXPRESSION_RESULT)
    "lhs.extMetaLen == lhs.extMetaLen" is always true regardless of
    the values of its operands because those operands are identical. This
    occurs as the logical second operand of "&&".

Change-Id: I87529e6a3801358bafe265869125c2eff2d40587
Reviewed-on: http://review.couchbase.org/70168
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
src/item.cc

index c3df0df..87338b5 100644 (file)
@@ -501,7 +501,7 @@ std::ostream& operator<<(std::ostream& os, const ItemMetaData& md) {
 
 bool operator==(const Blob& lhs, const Blob& rhs) {
     return (lhs.size == rhs.size) &&
-           (lhs.extMetaLen == lhs.extMetaLen) &&
+           (lhs.extMetaLen == rhs.extMetaLen) &&
            (lhs.age == rhs.age) &&
            (memcmp(lhs.data, rhs.data, lhs.size) == 0);
 }