9856 - [malloc] memory allocation problems - netw[eo]rk

Status: VERIFIED FIXED

(Core :: Networking, defect, P3)

Description

[chris hofmann]

s:/mozilla/netwerk/protocol/http/src/nsHTTPChannel.cpp
   Deref-error: "cookieTest", line 422
s:/mozilla/netwerk/test/TestFileInput.cpp
   Deref-error: "reader", line 266
   Deref-error: "reader", line 270
   Deref-error: "reader", line 329
   Deref-error: "reader", line 333
s:/mozilla/network/cache/nu/src/nsDiskModule.cpp
   Deref-error: "pTemp", line 286
   Deref-error: "pTempObj", line 533
   Deref-error: "pTempObj", line 535
s:/mozilla/network/cache/nu/tests/cb/cbView.cpp
   Deref-error: "pCO", line 123
   Deref-error: "pCO", line 124
   Deref-error: "pCO", line 125
   Deref-error: "pDM", line 126
   Deref-error: "ms", line 173
   Deref-error: "ms", line 174
   Deref-error: "ms", line 178
   Deref-error: "ms", line 179
s:/mozilla/network/module/nsNetFile.cpp
   Deref-error: "theFile", line 824
   Deref-error: "theFile", line 827
s:/mozilla/network/protocol/imap4/imaphier.cpp
   Deref-error: "nodeTokens", line 53
   Deref-error: "nodeTokens", line 54
s:/mozilla/network/protocol/sockstub/sockstub.cpp
   Deref-error: "theUrlToSocketHashtable", line 163
   Deref-error: "theUrlToSocketHashtable", line 172
   Deref-error: "theUrlToSocketHashtable", line 182
   Deref-error: "theUrlToSocketHashtable", line 196

Comment 1

[Warren Harris]

This one's in an #if 0, but Gagan should fix it if he's intending to turn this
on:

s:/mozilla/netwerk/protocol/http/src/nsHTTPChannel.cpp
   Deref-error: "cookieTest", line 422

The ones in TestFileInput.cpp look fine to me.

Gagan should also look at the ones in the cache/nu directory if we're going to
reuse that stuff.

Comment 2

[Gagan]

the cookie stuff is judson.. and if I understand this, its safe to remove. But
I'd let Jud comment on that.

Comment 3

[leger]

Changing all Networking Library/Browser bugs to Networking-Core component for
Browser.

Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do
this in a bulk change.  If this happens, I will fix. ;-)

Comment 4

[Judson Valeski]

cookieTest is gone, and the TestFileInput probs are being released.

Comment 5

[leger]

Bulk move of all Networking-Core (to be deleted component) bugs to new
Networking component.

Comment 6

[benc]

VERIFIED:
we have new processes to capture and address these issues.