Bug 1283060 - prevent null pointer dereference in logging::TreeInfo. r?surkov.alexander draft
authorAndi-Bogdan Postelnicu <bpostelnicu@mozilla.com>
Wed, 29 Jun 2016 14:06:31 +0300
changeset 382241 36205b94cc096d30f8b601c68e69929a126f22d0
parent 381966 e45890951ce77c3df05575bd54072b9f300d77b0
child 524144 1a743b0927d2c4cc3425d23c4039fe7e8c426f32
push id21668
push userbmo:bpostelnicu@mozilla.com
push dateWed, 29 Jun 2016 11:06:59 +0000
reviewerssurkov.alexander
bugs1283060
milestone50.0a1
Bug 1283060 - prevent null pointer dereference in logging::TreeInfo. r?surkov.alexander MozReview-Commit-ID: KhNGsH8MRVz
accessible/base/Logging.cpp
--- a/accessible/base/Logging.cpp
+++ b/accessible/base/Logging.cpp
@@ -643,17 +643,17 @@ logging::TreeInfo(const char* aMsg, uint
 void
 logging::TreeInfo(const char* aMsg, uint32_t aExtraFlags,
                   const char* aMsg1, Accessible* aAcc,
                   const char* aMsg2, nsINode* aNode)
 {
   if (IsEnabledAll(logging::eTree | aExtraFlags)) {
     MsgBegin("TREE", "%s; doc: %p", aMsg, aAcc ? aAcc->Document() : nullptr);
     AccessibleInfo(aMsg1, aAcc);
-    Accessible* acc = aAcc->Document()->GetAccessible(aNode);
+    Accessible* acc = aAcc ? aAcc->Document()->GetAccessible(aNode) : nullptr;
     if (acc) {
       AccessibleInfo(aMsg2, acc);
     }
     else {
       Node(aMsg2, aNode);
     }
     MsgEnd();
   }