Project

General

Profile

Actions

Bug #4818

open

murmurHash3 of uint32_t is not endian safe

Added by Anonymous over 5 years ago. Updated over 1 year ago.

Status:
Code review
Priority:
Normal
Assignee:
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

murmurHash3 of an uint32_t value computes the hash of the in-memory 4-byte buffer. If this is part of a data structure that is sent to another computer with a difference between big endian and little endian, then they won't compute the same hash.
https://github.com/named-data/PSync/blob/0cf4b600e91455ee07c38eb22876aa6b653bc14b/PSync/detail/util.cpp#L103

Maybe this should use endian::native_to_little, similar to the TLV encoder (Intel chips use little endian):
https://github.com/named-data/ndn-cxx/blob/master/ndn-cxx/encoding/encoder.cpp#L151


Related issues 1 (1 open0 closed)

Related to PSync - Task #4659: Look into using newer and better hash functionCode reviewJunxiao Shi

Actions
Actions

Also available in: Atom PDF