-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63422/
-----------------------------------------------------------

(Updated Jan. 3, 2018, 10:41 p.m.)


Review request for mesos and Greg Mann.


Summary (updated)
-----------------

Added os::eraseenv to clear the old environment value.


Bugs: MESOS-8140
    https://issues.apache.org/jira/browse/MESOS-8140


Repository: mesos


Description (updated)
-------

On Linux, updating the environment with unsetenv(3) does
not change the value that other processes can inspect
(e.g. through `/proc/$pid/environ`). We have to overwrite
the existing value (if any) prior to calling unsetenv(3)
if we want to ensure that the value is really gone.


Diffs (updated)
-----

  3rdparty/stout/include/stout/posix/os.hpp 
7427bd792368fbacafb23593dfb9213618fdcddf 


Diff: https://reviews.apache.org/r/63422/diff/2/

Changes: https://reviews.apache.org/r/63422/diff/1-2/


Testing
-------

make check (Fedora 26)


Thanks,

James Peach

Reply via email to