Closed
Bug 50878
Opened 24 years ago
Closed 21 years ago
MLK: javascript urls that assert
Categories
(Core :: Networking, defect, P3)
Core
Networking
Tracking
()
RESOLVED
INVALID
Future
People
(Reporter: bruce, Assigned: gagan)
Details
(Keywords: memory-leak, Whiteboard: [nsbeta3-])
Attachments
(2 files)
3.49 KB,
patch
|
Details | Diff | Splinter Review | |
3.49 KB,
patch
|
Details | Diff | Splinter Review |
Build from last night. Run a javascript: url that asserts. Got this leak. MLK: 11 bytes leaked at 0x21c5340 This memory was allocated from: malloc [rtlib.o] PL_strdup [strdup.c:30] nsCRT::strdup(const char*) [nsCRT.h:172] nsSimpleURI::GetPath(char**) [nsSimpleURI.cpp:216] nsJSThunk::Open(char**,int*) [nsJSProtocolHandler.cpp:301] nsFileTransport::Process() [nsFileTransport.cpp:387] PR_LOG(gFileTransportLog, PR_LOG_DEBUG, ("nsFileTransport: OPEN_FOR_READ [this=%x %s]", this, mStreamName.GetBuffer())); => mStatus = mStreamIO->Open(&mContentType, &mTotalAmount); if (mListener) { nsresult rv = mListener->OnStartRequest(this, mContext); // always send the start notification if (NS_SUCCEEDED(mStatus)) nsFileTransport::Run() [nsFileTransport.cpp:361] nsThreadPoolRunnable::Run() [nsThread.cpp:689] nsThread::Main(void*) [nsThread.cpp:84] _pt_root [ptthread.c:198] _thread_start [libthread.so.1]
Reporter | ||
Updated•24 years ago
|
Hmmm... for some reason this page timedout the first time. Anyway, I am not sure this could make it for beta3 per our new rules. But this should get picked up on the next cycle.
Keywords: patch
Whiteboard: [nsbeta3-]
This is a really old bug, can someone confirm that it still happens or WFM?
Comment 7•21 years ago
|
||
Can't find nsFileTransport.cpp in lxr, maybe removed or moved to some other interface?
RESOLVED/INVALID: Mikael: thanks for pointing that out. Can you VERIFY this bug?
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•