13290 - MLK: nsScriptSecurityManager?

Status: VERIFIED FIXED

(Core :: Security, defect, P3)

Description

[Bruce Mitchener]

Build from today.  Appears to be happening a lot.  Is this because something
larger holding on to this is being leaked?  if this is related to the webshell,
then let's mark this as depending on bug #13283

[W] MLK: Memory leak of 60 bytes from 1 block allocated in
nsStdURL::Create(nsISupports *,nsID const&,void * *)
        Distribution of leaked blocks
                60 bytes from 1 block of 60 bytes (0x097f19d8) allocation number
182455
        Allocation location
            new(UINT)      [new.cpp:23]
            nsStdURL::Create(nsISupports *,nsID const&,void * *)
[nsStdURL.cpp:627]
            nsGenericFactory::CreateInstance(nsISupports *,nsID const&,void * *)
[nsGenericFactory.cpp:37]
            nsComponentManagerImpl::CreateInstance(nsID const&,nsISupports
*,nsID const&,void * *) [nsComponentManager.cpp:1203]
            nsScriptSecurityManager::GetSitePolicy(char const*)
[nsScriptSecurityManager.cpp:421]
            nsScriptSecurityManager::AddSecPolicyPrefix(JSContext *,char *)
[nsScriptSecurityManager.cpp:394]
            nsScriptSecurityManager::GetSecurityLevel(JSContext *,char *,int)
[nsScriptSecurityManager.cpp:355]
            nsScriptSecurityManager::CheckScriptAccess(nsIScriptContext *,void
*,char const*,int,int *) [nsScriptSecurityManager.cpp:93]
            GetWindowProperty [nsJSWindow.cpp:275]
            js_Interpret   [jsinterp.c:2184]
            js_Invoke      [jsinterp.c:670]
            js_InternalCall [jsinterp.c:747]
            JS_CallFunction [jsapi.c:2611]
            nsJSDOMEventListener::HandleEvent(nsIDOMEvent *)
[nsJSDOMEventListener.cpp:79]
            nsEventListenerManager::HandleEvent(nsIPresContext&,nsEvent
*,nsIDOMEvent * *,UINT,nsEventStatus&)

Comment 1

[Norris Boyd]

This method needs cleanup.

Comment 2

[Norris Boyd]

Removed offending code.

Comment 3

[John Unruh]

Verified.

Comment 4

[leger]

Bulk moving all Browser Security bugs to new Security: General component.  The 
previous Security component for Browser will be deleted.