mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Peach <>
Subject Re: Review Request 58305: Add some parameter validation to ReRegisterSlaveMessage.
Date Wed, 19 Apr 2017 22:05:00 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated April 19, 2017, 10:05 p.m.)

Review request for mesos, Mesos Reviewbot and Neil Conway.

Bugs: MESOS-7372

Repository: mesos


The ReRegisterSlaveMessage message sends a number of fields which have
internal consistency requirements. Add some simple validation checks
to ensure that we have a minimally consistent re-registration request
before proceeding.

Diffs (updated)

  src/master/master.cpp 52de2f91bdacf46f913c27382ad50b4f278ad297 
  src/master/validation.hpp d96287de73ddb30ae2ed841c1b910b0ac6cfa74e 
  src/master/validation.cpp 3f70875484bbd856ac79a7d6070ac313d69782fa 
  src/tests/master_validation_tests.cpp 555380870ae115004312cfbe9f145faa92049030 




Make check (Fedora 25). Internal fuzzing.


James Peach

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message