MLK: javascript urls that assert

RESOLVED INVALID

Status

()

P3
normal
RESOLVED INVALID
18 years ago
15 years ago

People

(Reporter: bruce, Assigned: gagan)

Tracking

({memory-leak})

Trunk
Future
memory-leak
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta3-])

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
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

18 years ago
Keywords: mlk, nsbeta3
(Assignee)

Comment 1

18 years ago
Created attachment 15332 [details] [diff] [review]
patch for mem leak
(Assignee)

Comment 2

18 years ago
Created attachment 15334 [details] [diff] [review]
patch for mem leak
(Assignee)

Comment 3

18 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-]
(Assignee)

Updated

18 years ago
Target Milestone: --- → Future

Comment 4

18 years ago
mass move, v2.
qa to me.
QA Contact: tever → benc

Updated

17 years ago
Blocks: 92580

Updated

17 years ago
No longer blocks: 92580

Comment 5

15 years ago
-> defaults

Comment 6

15 years ago
This is a really old bug, can someone confirm that it still happens or WFM?

Comment 7

15 years ago
Can't find nsFileTransport.cpp in lxr, maybe removed or moved to some other
interface?

Comment 8

15 years ago
RESOLVED/INVALID:

Mikael: thanks for pointing that out. Can you VERIFY this bug?
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.