nsSocketBOS::Write does not initialize return value

VERIFIED FIXED in mozilla0.9

Status

()

Core
Networking
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Robert Ginda, Assigned: Darin Fisher)

Tracking

Trunk
mozilla0.9
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

17 years ago
Recent regression, here's a patch that seems to fix problems this caused for
chatzilla.

Index: nsSocketTransport.cpp
===================================================================
RCS file: /cvsroot/mozilla/netwerk/base/src/nsSocketTransport.cpp,v
retrieving revision 1.198
diff -u -r1.198 nsSocketTransport.cpp
--- nsSocketTransport.cpp       2001/04/10 06:57:01     1.198
+++ nsSocketTransport.cpp       2001/04/14 23:43:30
@@ -2080,7 +2080,7 @@
 NS_IMETHODIMP
 nsSocketBOS::Write(const char *aBuf, PRUint32 aCount, PRUint32 *aBytesWritten)
 {
-    nsresult rv;
+    nsresult rv = NS_OK;
     PRInt32 total = 0;
     PRInt32 written;

Comment 1

17 years ago
-->darin
Assignee: neeti → darin
(Assignee)

Comment 2

17 years ago
rginda: thanks for catching this!
Status: NEW → ASSIGNED
Keywords: nsbeta1
Target Milestone: --- → mozilla0.9
(Assignee)

Updated

17 years ago
Keywords: patch
(Assignee)

Comment 3

17 years ago
fix checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Updated

17 years ago
QA Contact: tever → bbaetz
VERIFIED fixed in CVS.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.