mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <>
Subject Review Request 65305: Fixed a libprocess crash when tests clear the authorization callbacks.
Date Wed, 24 Jan 2018 00:54:14 GMT

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

Review request for mesos, Alexander Rukletsov, Alexander Rojas, and Joseph Wu.

Bugs: MESOS-8453

Repository: mesos


The authorization callbacks are not mutated in a thread-safe manner:
a `Process` may be calling a callback while the callbacks are being
deleted by another thread.

The fix here for now is to make the mutation and access synchronized
under a mutex. The mutex is non-recursive, which means that the
callbacks must not be cleared or re-set from within a callback. This
is already a requirement, since a callback calling in to clear or
re-set the callbacks is thereby deleting itself!

Some TODOs were left for performance improvements. Since the HTTP
likely has much more significant bottlenecks, I left these as TODOs.


  3rdparty/libprocess/src/process.cpp 2126c272a69bfa53b4b3cbbb1a55a3f81a5da8ad 



make check


Benjamin Mahler

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