diff --git a/extensions/spellcheck/src/mozEnglishWordUtils.cpp b/extensions/spellcheck/src/mozEnglishWordUtils.cpp index 644508963..6d0856f3e 100644 --- a/extensions/spellcheck/src/mozEnglishWordUtils.cpp +++ b/extensions/spellcheck/src/mozEnglishWordUtils.cpp @@ -164,7 +164,7 @@ NS_IMETHODIMP mozEnglishWordUtils::FindNextWord(const char16_t *word, uint32_t l // before we spend more time looking to see if the word is a url, look for a url identifer // and make sure that identifer isn't the last character in the word fragment. - if ( (*p == ':' || *p == '@' || *p == '.') && p < endbuf - 1) { + if ((p < endbuf - 1) && (*p == ':' || *p == '@' || *p == '.')) { // ok, we have a possible url...do more research to find out if we really have one // and determine the length of the url so we can skip over it. diff --git a/netwerk/protocol/http/nsHttpChannelAuthProvider.cpp b/netwerk/protocol/http/nsHttpChannelAuthProvider.cpp index f520bef76..8e7539c25 100644 --- a/netwerk/protocol/http/nsHttpChannelAuthProvider.cpp +++ b/netwerk/protocol/http/nsHttpChannelAuthProvider.cpp @@ -439,13 +439,11 @@ nsHttpChannelAuthProvider::PrepareForAuthentication(bool proxyAuth) // We need to remove any Proxy_Authorization header left over from a // non-request based authentication handshake (e.g., for NTLM auth). - nsAutoCString contractId; - contractId.Assign(NS_HTTP_AUTHENTICATOR_CONTRACTID_PREFIX); - contractId.Append(mProxyAuthType); - nsresult rv; - nsCOMPtr precedingAuth = - do_GetService(contractId.get(), &rv); + nsCOMPtr precedingAuth; + nsCString proxyAuthType; + rv = GetAuthenticator(mProxyAuthType.get(), proxyAuthType, + getter_AddRefs(precedingAuth)); if (NS_FAILED(rv)) return rv; diff --git a/netwerk/sctp/datachannel/DataChannel.cpp b/netwerk/sctp/datachannel/DataChannel.cpp index f00eea825..cc59f1661 100644 --- a/netwerk/sctp/datachannel/DataChannel.cpp +++ b/netwerk/sctp/datachannel/DataChannel.cpp @@ -232,6 +232,7 @@ DataChannelConnection::Destroy() MOZ_ASSERT(mSTS); ASSERT_WEBRTC(NS_IsMainThread()); + mListener = nullptr; // Must do this in Destroy() since we may then delete this object. // Do this before dispatching to create a consistent ordering of calls to // the SCTP stack.