--- a/uriloader/exthandler/nsExternalHelperAppService.cpp
+++ b/uriloader/exthandler/nsExternalHelperAppService.cpp
@@ -1481,17 +1481,17 @@ nsExternalHelperAppService::GetProtocolH
   // XXX enterprise customers should be able to turn this support off with a
   // single master pref (maybe use one of the "exposed" prefs here?)
 
   // nsIMIMEInfo is a superset of nsIHandlerInfo.  Furthermore, nsMimeInfoImpl
   // and subclasses have lots of good platform specific-knowledge of local
   // applications which we might need later.  For now, just use nsMIMEInfoImpl
   // instead of implementating a separate nsIHandlerInfo object.
   *aHandlerInfo = GetProtocolInfoFromOS(aScheme).get();
-  if (!aHandlerInfo) {
+  if (!(*aHandlerInfo)) {
     // Either it knows nothing, or we ran out of memory
     return NS_ERROR_FAILURE;
   }
 
   nsresult rv = FillProtoInfoForSchemeFromDS(aScheme, *aHandlerInfo);     
   if (NS_ERROR_NOT_AVAILABLE == rv) {
     // We don't know it, so we always ask the user.  By the time we call this
     // method, we already have checked if we should open this protocol and ask