Bug 1310127 - Part 8: Use MOZ_MUST_USE in netwerk/protocol/http r?mkaply draft
authorWei-Cheng Pan <wpan@mozilla.com>
Tue, 20 Dec 2016 17:19:46 +0800
changeset 494455 b0c1cf913ccc1721b0f4b6ec8b1def308f217817
parent 494454 3f7549a732127d038f07e044489160a3f290b2e9
child 494456 90e1b2149a45fdd666e68dd37f7dd979ef026f4d
push id48029
push userbmo:wpan@mozilla.com
push dateTue, 07 Mar 2017 03:35:29 +0000
reviewersmkaply
bugs1310127
milestone54.0a1
Bug 1310127 - Part 8: Use MOZ_MUST_USE in netwerk/protocol/http r?mkaply MozReview-Commit-ID: Cd7rL6rpy6e
extensions/pref/autoconfig/src/nsAutoConfig.cpp
--- a/extensions/pref/autoconfig/src/nsAutoConfig.cpp
+++ b/extensions/pref/autoconfig/src/nsAutoConfig.cpp
@@ -131,18 +131,18 @@ nsAutoConfig::OnStopRequest(nsIRequest *
                                        static_cast<uint32_t>(aStatus)));
         return readOfflineFile();
     }
 
     // Checking for the http response, if failure go read the failover file.
     nsCOMPtr<nsIHttpChannel> pHTTPCon(do_QueryInterface(request));
     if (pHTTPCon) {
         uint32_t httpStatus;
-        pHTTPCon->GetResponseStatus(&httpStatus);
-        if (httpStatus != 200) 
+        rv = pHTTPCon->GetResponseStatus(&httpStatus);
+        if (NS_FAILED(rv) || httpStatus != 200)
         {
             MOZ_LOG(MCD, LogLevel::Debug, ("mcd http request failed with status %x\n", httpStatus));
             return readOfflineFile();
         }
     }
     
     // Send the autoconfig.jsc to javascript engine.