From fc4cd2c7f1762df3f1b99802cb69a23f00c9afd5 Mon Sep 17 00:00:00 2001 From: Justin Lebar Date: Mon, 16 May 2011 16:36:12 -0400 Subject: [PATCH] Bug 656991 - Followup. Use JS_free instead of free. r=bz --HG-- extra : rebase_source : 00e5cc27f067a191e7a834852faee8c788feb061 --- dom/base/nsStructuredCloneContainer.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/dom/base/nsStructuredCloneContainer.cpp b/dom/base/nsStructuredCloneContainer.cpp index 848991486393..b8ce3d0bd6ec 100644 --- a/dom/base/nsStructuredCloneContainer.cpp +++ b/dom/base/nsStructuredCloneContainer.cpp @@ -97,7 +97,10 @@ nsStructuredCloneContainer::InitFromVariant(nsIVariant *aData, JSContext *aCx) if (!mData) { mSize = 0; mVersion = 0; - free(jsBytes); + + // FIXME This should really be js::Foreground::Free, but that's not public. + JS_free(aCx, jsBytes); + return NS_ERROR_FAILURE; } else { @@ -105,7 +108,9 @@ nsStructuredCloneContainer::InitFromVariant(nsIVariant *aData, JSContext *aCx) } memcpy(mData, jsBytes, mSize); - free(jsBytes); + + // FIXME Similarly, this should be js::Foreground::free. + JS_free(aCx, jsBytes); return NS_OK; }