Closed Bug 76064 Opened 23 years ago Closed 23 years ago

nsSocketBOS::Write does not initialize return value

Categories

(Core :: Networking, defect)

Product:
Core
Component:
Networking
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla0.9

People

(Reporter: rginda, Assigned: darin.moz)

Details

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;
-->darin
Assignee: neeti → darin
rginda: thanks for catching this!
Status: NEW → ASSIGNED
Keywords: nsbeta1
Target Milestone: --- → mozilla0.9
Keywords: patch
fix checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
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.