You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2016/04/25 23:32:16 UTC
[23/50] mesos git commit: Fixed a NULL pointer dereference bug in
Slave.
Fixed a NULL pointer dereference bug in Slave.
This fixes a `NULL` pointer dereference erroneously added as part of
introducing an output operator for the `Executor` struct in
https://reviews.apache.org/r/39569.
Review: https://reviews.apache.org/r/42605/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/cbc4afed
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/cbc4afed
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/cbc4afed
Branch: refs/heads/0.26.x
Commit: cbc4afed99b0d43f9c9c2ae14e82b285defc7372
Parents: bfe6110
Author: Anand Mazumdar <ma...@gmail.com>
Authored: Fri Jan 22 01:37:47 2016 -0800
Committer: Michael Park <mp...@apache.org>
Committed: Fri Feb 26 20:59:05 2016 -0800
----------------------------------------------------------------------
src/slave/slave.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/cbc4afed/src/slave/slave.cpp
----------------------------------------------------------------------
diff --git a/src/slave/slave.cpp b/src/slave/slave.cpp
index ac2d0e0..05b4b4f 100644
--- a/src/slave/slave.cpp
+++ b/src/slave/slave.cpp
@@ -2111,7 +2111,7 @@ void Slave::schedulerMessage(
Executor* executor = framework->getExecutor(executorId);
if (executor == NULL) {
- LOG(WARNING) << "Dropping message for executor " << *executor
+ LOG(WARNING) << "Dropping message for executor " << executorId
<< " because executor does not exist";
metrics.invalid_framework_messages++;
return;