[tor-commits] [tor-browser/tor-browser-78.14.0esr-10.5-1] Bug 1650281 - P1: Widen `gCombinedSizes` once the buffers grow r=gerald

sysrqb at torproject.org sysrqb at torproject.org
Sat Sep 4 01:29:00 UTC 2021


commit e4d2c98010698e7965aff4f475c6f49abfa2aa45
Author: Chun-Min Chang <chun.m.chang at gmail.com>
Date:   Tue Jul 21 23:38:57 2020 +0000

    Bug 1650281 - P1: Widen `gCombinedSizes` once the buffers grow r=gerald
    
    The `gCombinedSizes` need to be enlarged once the inner buffer within
    `MemoryBlockCache` grows. Otherwise, when the `MemoryBlockCache` is
    released, subtracting the buffer-size of the `MemoryBlockCache` from
    `gCombinedSizes` lead to a underflow.
    
    Differential Revision: https://phabricator.services.mozilla.com/D84273
---
 dom/media/MemoryBlockCache.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/dom/media/MemoryBlockCache.cpp b/dom/media/MemoryBlockCache.cpp
index 2d31119dca0a..bf073e6769d0 100644
--- a/dom/media/MemoryBlockCache.cpp
+++ b/dom/media/MemoryBlockCache.cpp
@@ -114,6 +114,10 @@ bool MemoryBlockCache::EnsureBufferCanContain(size_t aContentLength) {
     // possibly bypass some future growths that would fit in this new capacity.
     mBuffer.SetLength(capacity);
   }
+  const size_t newSizes = gCombinedSizes += (extra + extraCapacity);
+  LOG("EnsureBufferCanContain(%zu) - buffer size %zu + requested %zu + bonus "
+      "%zu = %zu; combined sizes %zu",
+      aContentLength, initialLength, extra, extraCapacity, capacity, newSizes);
   mHasGrown = true;
   return true;
 }





More information about the tor-commits mailing list