9861 - [Webshell] [malloc] memory allocation problems - webshell

Status: RESOLVED FIXED

(Core Graveyard :: Tracking, defect, P3)

Description

[chris hofmann]

[malloc] memory allocation problems - webshell

check bug 8227 for more details on what needs to be done to this code

s:/mozilla/webshell/embed/ActiveX/tests/IEPatcher/IEPatcherDlg.cpp
   Deref-error: "pData", line 515
   Deref-error: "pData", line 516
   Deref-error: "pData", line 517
   Deref-error: "pData", line 520
s:/mozilla/webshell/embed/ngprefs/win/ngprefs.cpp
   Deref-error: "pMailNews", line 958
   Deref-error: "pMailNews", line 962
   Deref-error: "pMailNews", line 964
   Deref-error: "pMailNews", line 975
   Deref-error: "pMailNews", line 977
   Deref-error: "pMailNews", line 986
   Deref-error: "pAppearance", line 1248
   Deref-error: "pAppearance", line 1252
   Deref-error: "pAppearance", line 1254
   Deref-error: "pAppearance", line 1265
   Deref-error: "pAppearance", line 1267
   Deref-error: "pAppearance", line 1276
s:/mozilla/webshell/tests/viewer/nsBeOSMain.cpp
   Deref-error: "app", line 95
   Deref-error: "app", line 96
s:/mozilla/webshell/tests/viewer/nsBrowserWindow.cpp
   Deref-error: "bw", line 2202
   Deref-error: "bw", line 2203
   Deref-error: "bw", line 2205
s:/mozilla/webshell/tests/viewer/nsMotifMenu.cpp
   Deref-error: "cbs", line 85
   Deref-error: "cbs", line 86
s:/mozilla/webshell/tests/viewer/nsOS2Main.cpp
   Deref-error: "app", line 76
   Deref-error: "app", line 77
s:/mozilla/webshell/tests/viewer/nsViewerApp.cpp
   Deref-error: "gWorkList", line 1005
s:/mozilla/webshell/tests/viewer/nsWinMain.cpp
   Deref-error: "app", line 101
   Deref-error: "app", line 102
   Deref-error: "app", line 115
   Deref-error: "app", line 116
s:/mozilla/webshell/tests/viewer/nsXlibMain.cpp
   Deref-error: "app", line 68
   Deref-error: "app", line 69

Comment 1

[Scott Collins]

(Mass-) Re-assigning [Webshell] bugs to travis, on the advice of dp, et al.  Many
of these may be invalid in the new world of the re-written webshell, but he
certainly needs to be aware of these issues.  Hope this helps you, travis.

Comment 2

[chris hofmann]

m12

Comment 3

[don]

Move to M13 and re-assess once basic Webshell re-design is complete.

Comment 4

[don]

Move to M14.

Comment 5

[travis]

I don't see anything needing to be done in this list in the base webshell....  
I don't see why I have this bug therefore, someone else needs to take it.  Is 
this still an issue?  Fixed?

Comment 6

[don]

Move to M16 ...

Comment 7

[travis]

Marking fixed based on my earlier comments.