[X2Go-Commits] [pale-moon] 77/102: Get RefPtr to transaction before using it. (DiD)

git-admin at x2go.org git-admin at x2go.org
Mon Feb 25 23:25:54 CET 2019


This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch upstream/28.4.0
in repository pale-moon.

commit 2b1ccf227b8f409e0572119018c7615b877a6612
Author: wolfbeast <mcwerewolf at wolfbeast.com>
Date:   Thu Feb 7 12:39:42 2019 +0100

    Get RefPtr to transaction before using it. (DiD)
    
    Avoid potential use-after-free by getting a RefPtr to a transaction
    before calling content code that could cause its de-allocation.
---
 dom/indexedDB/ActorsChild.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/dom/indexedDB/ActorsChild.cpp b/dom/indexedDB/ActorsChild.cpp
index 3e8f973..c4fcceb 100644
--- a/dom/indexedDB/ActorsChild.cpp
+++ b/dom/indexedDB/ActorsChild.cpp
@@ -3456,6 +3456,8 @@ BackgroundCursorChild::RecvResponse(const CursorResponse& aResponse)
 
   RefPtr<IDBCursor> cursor;
   mStrongCursor.swap(cursor);
+  
+  RefPtr<IDBTransaction> transaction = mTransaction;
 
   switch (aResponse.type()) {
     case CursorResponse::Tnsresult:
@@ -3486,7 +3488,7 @@ BackgroundCursorChild::RecvResponse(const CursorResponse& aResponse)
       MOZ_CRASH("Should never get here!");
   }
 
-  mTransaction->OnRequestFinished(/* aActorDestroyedNormally */ true);
+  transaction->OnRequestFinished(/* aActorDestroyedNormally */ true);
 
   return true;
 }

--
Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/pale-moon.git


More information about the x2go-commits mailing list