Bug 1250540 - fix string-parsing regression that made screen share whitelisting stop working. draft
authorJan-Ivar Bruaroey <jib@mozilla.com>
Tue, 23 Feb 2016 11:30:49 -0500
changeset 333384 d1e729587d560b86f63afe228567ee217326e887
parent 332162 69ec3dc408a2a720cb2b8210fea33e3504aeec22
child 333924 4320434c04ac48e0604bffb9233a513bab4e543d
push id11350
push userjbruaroey@mozilla.com
push dateTue, 23 Feb 2016 16:31:46 +0000
bugs1250540
milestone47.0a1
Bug 1250540 - fix string-parsing regression that made screen share whitelisting stop working. MozReview-Commit-ID: AKux0nBVBEF
dom/media/MediaManager.cpp
--- a/dom/media/MediaManager.cpp
+++ b/dom/media/MediaManager.cpp
@@ -219,17 +219,17 @@ HostHasPermission(nsIURI &docURI)
   */
   do {
     end = domainWhiteList.FindChar(',', begin);
     if (end == (uint32_t)-1) {
       // Last or only domain name in the comma separated list
       end = domainWhiteList.Length();
     }
 
-    rv = idnService->ConvertUTF8toACE(Substring(domainWhiteList, begin, end),
+    rv = idnService->ConvertUTF8toACE(Substring(domainWhiteList, begin, end - begin),
                                       domainName);
     if (NS_SUCCEEDED(rv)) {
       if (HostInDomain(hostName, domainName)) {
         return true;
       }
     } else {
       NS_WARNING("Failed to convert UTF-8 host to ASCII");
     }