Closed
Bug 119220
Opened 23 years ago
Closed 23 years ago
Occurances of uninitialized variables being used before being set.
Categories
(Core :: Networking, defect)
Tracking
()
VERIFIED
FIXED
mozilla0.9.9
People
(Reporter: mozilla-bugs, Assigned: neeti)
References
Details
Attachments
(1 file)
(deleted),
patch
|
dougt
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
From http://tinderbox.mozilla.org/SeaMonkey/warn1010629440.4199.html:
netwerk/base/public/nsReadLine.h:63
`char eolchar' might be used uninitialized in this function
netwerk/base/src/nsFileTransport.cpp:932
`PRUint32 total' might be used uninitialized in this function
netwerk/base/src/nsStandardURL.cpp:1641
`nsresult rv' might be used uninitialized in this function
netwerk/streamconv/converters/nsFTPDirListingConv.cpp:952
`long unsigned int size' might be used uninitialized in this function
netwerk/test/TestPageLoad.cpp:198
`nsresult rv' might be used uninitialized in this function
netwerk/test/TestSocketTransport.cpp:421
`nsresult rv' might be used uninitialized in this function
P.S. This is a "next generation" bug 59661
The above patch fixes all the warnings, except
netwerk/base/src/nsStandardURL.cpp:1641
`nsresult rv' might be used uninitialized in this function
which was fixed by bug 119330
darin,doug: could you review the patch.
Comment 4•23 years ago
|
||
Comment on attachment 69283 [details] [diff] [review]
patch which fixes these warnings
sr=darin
thx for cleaning these up
Attachment #69283 -
Flags: superreview+
Comment 5•23 years ago
|
||
Comment on attachment 69283 [details] [diff] [review]
patch which fixes these warnings
even if we have the wrong values defaulted, we are at least consistent. :-)
Attachment #69283 -
Flags: review+
fix checked in
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•