summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>2013-05-29 15:37:31 +0200
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>2013-05-31 19:15:23 +0200
commit578e292b74331ac8f4c2fcfd464edb7d83a6a02f (patch)
tree0fa7c0f358e49784154c81605ab13aa583f3ad7e
parentc0e3d04bb83361017c2f766d013161aa659264e8 (diff)
downloadceph-578e292b74331ac8f4c2fcfd464edb7d83a6a02f.tar.gz
mds/Migrator.cc: fix dereference after null check
Add assert for 'le' to fix: CID 716923 (#1 of 1): Dereference after null check (FORWARD_NULL) var_deref_model: Passing null pointer "le" to function "LogEvent::get_start_off() const", which dereferences it. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
-rw-r--r--src/mds/Migrator.cc1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mds/Migrator.cc b/src/mds/Migrator.cc
index 92962424e46..9fe5451432e 100644
--- a/src/mds/Migrator.cc
+++ b/src/mds/Migrator.cc
@@ -2518,6 +2518,7 @@ int Migrator::decode_import_dir(bufferlist::iterator& blp,
}
else if (icode == 'I') {
// inode
+ assert(le);
decode_import_inode(dn, blp, oldauth, ls, le->get_start_off(), cap_imports, updated_scatterlocks);
}