----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37903/#review99144 -----------------------------------------------------------
Patch looks great! Reviews applied: [37903] All tests passed. - Mesos ReviewBot On Sept. 15, 2015, 9:10 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37903/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2015, 9:10 p.m.) > > > Review request for mesos. > > > Bugs: MESOS-3328 > https://issues.apache.org/jira/browse/MESOS-3328 > > > Repository: mesos > > > Description > ------- > > The previous coding would try to shift a uint32_t value 32 bits to the left; > per > C++ spec, this yields undefined behavior. On my machine, this resulted in > treating a prefix of 0 as equivalent to a prefix of 32, which is obviously > wrong. > > Spotted via ubsan: see MESOS-3328. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/ip.hpp > 1ad119d54820e97497b1773518875be25ddbf98a > 3rdparty/libprocess/3rdparty/stout/tests/ip_tests.cpp > b0cbcb38cfcb923ec7c185bacf139ceb0a28924f > > Diff: https://reviews.apache.org/r/37903/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Neil Conway > >