mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jiang Yan Xu <>
Subject Review Request 55447: Added sanity checks on IDs and roles before creating directories.
Date Thu, 12 Jan 2017 09:48:25 GMT

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

Review request for mesos, Benjamin Mahler and James Peach.

Bugs: MESOS-6866

Repository: mesos


The context for adding these is MESOS-6866. Addmittedly these checks only protect against
a (small) subset of potential attacks by clients spoofing as the master and we need MESOS-6903
to truely fix the issue. However instead of ad-hoc bandaids, these could be considered good
programming practices: CHECK preconditions before performing (potentially dangerous) actions
so I think this is still reasonable.


  src/slave/paths.cpp 8792cee43d94e7b0bbd7b80aebbe501236244621 



make check.


Jiang Yan Xu

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