Bug 1462316 - fix crash in nsAbView::OnItemAdded(). r=mkmelin draft comm/default
authorJorg K <jorgk@jorgk.com>
Sat, 06 Oct 2018 21:06:14 +0200
branchcomm/default
changeset 565260 8f8b0933b611ff24ddecb640ae012ef2cdad8c97
parent 565259 dd0f997fc9e38f6d556476fdf7c7e9c2d5c69322
child 565261 4203a87eb7d7caeeac18b419c1d64951e702ecb3
push id2
push usermkmelin@iki.fi
push dateMon, 08 Oct 2018 19:06:05 +0000
converted from8ea70a1b44eae6e9a7d83dd5c79cdca38aa60a9b
reviewersmkmelin
bugs1462316
Bug 1462316 - fix crash in nsAbView::OnItemAdded(). r=mkmelin
comm/mailnews/addrbook/src/nsAbView.cpp
--- a/comm/mailnews/addrbook/src/nsAbView.cpp
+++ b/comm/mailnews/addrbook/src/nsAbView.cpp
@@ -833,16 +833,19 @@ nsCString getQuery(nsCOMPtr<nsIAbDirecto
   if (searchBegin == kNotFound)
     return EmptyCString();
 
   return nsCString(Substring(uri, searchBegin));
 }
 
 NS_IMETHODIMP nsAbView::OnItemAdded(nsISupports *parentDir, nsISupports *item)
 {
+  if (!mDirectory)  // No address book selected.
+    return NS_OK;
+
   nsresult rv;
   nsCOMPtr <nsIAbDirectory> directory = do_QueryInterface(parentDir, &rv);
   NS_ENSURE_SUCCESS(rv,rv);
 
   bool isRemote = isDirectoryRemote(directory);
   // If the search is performed on All Address books, its possible that the LDAP
   // results start coming when mDirectory has changed (LDAP search works in an
   // asynchronous manner).