Updated Branches:
  refs/heads/master 137f4e86e -> 13b8a8d2c

Fixed a null pointer dereference in LevelDBStorage::initialize.


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/13b8a8d2
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/13b8a8d2
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/13b8a8d2

Branch: refs/heads/master
Commit: 13b8a8d2c852acb0968a55663af6e7e6a62d59d9
Parents: 137f4e8
Author: Benjamin Mahler <bmah...@twitter.com>
Authored: Wed Oct 30 11:58:24 2013 -0700
Committer: Benjamin Mahler <bmah...@twitter.com>
Committed: Wed Oct 30 11:58:24 2013 -0700

----------------------------------------------------------------------
 src/state/leveldb.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/13b8a8d2/src/state/leveldb.cpp
----------------------------------------------------------------------
diff --git a/src/state/leveldb.cpp b/src/state/leveldb.cpp
index 69d5714..49872b8 100644
--- a/src/state/leveldb.cpp
+++ b/src/state/leveldb.cpp
@@ -53,10 +53,10 @@ void LevelDBStorageProcess::initialize()
   if (!status.ok()) {
     // TODO(benh): Consider trying to repair the DB.
     error = Option<string>::some(status.ToString());
+  } else {
+    // TODO(benh): Conditionally compact to avoid long recovery times?
+    db->CompactRange(NULL, NULL);
   }
-
-  // TODO(benh): Conditionally compact to avoid long recovery times?
-  db->CompactRange(NULL, NULL);
 }
 
 

Reply via email to