Skip to content

Commit

Permalink
src: only memcmp if length > 0 in Buffer::Compare
Browse files Browse the repository at this point in the history
Both pointer arguments to memcmp are defined as non-null
and compiler optimizes upon that
  • Loading branch information
skomski committed Aug 27, 2015
1 parent 96a2b2d commit 5de6a6e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/node_buffer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -835,7 +835,8 @@ void Compare(const FunctionCallbackInfo<Value> &args) {

size_t cmp_length = MIN(obj_a_length, obj_b_length);

int32_t val = memcmp(obj_a_data, obj_b_data, cmp_length);
int val = cmp_length > 0 ? memcmp(obj_a_data, obj_b_data, cmp_length)
: 0;

// Normalize val to be an integer in the range of [1, -1] since
// implementations of memcmp() can vary by platform.
Expand Down
3 changes: 3 additions & 0 deletions test/parallel/test-buffer.js
Original file line number Diff line number Diff line change
Expand Up @@ -1137,6 +1137,9 @@ assert.equal(Buffer.compare(d, b), 1);
assert.equal(Buffer.compare(b, d), -1);
assert.equal(Buffer.compare(c, c), 0);

assert.equal(Buffer.compare(Buffer(0), Buffer(0)), 0);
assert.equal(Buffer.compare(Buffer(0), Buffer(1)), -1);
assert.equal(Buffer.compare(Buffer(1), Buffer(0)), 1);

assert.throws(function() {
var b = new Buffer(1);
Expand Down

0 comments on commit 5de6a6e

Please sign in to comment.