nsAStreamCopier::Process can use sourceCondition, sinkCondition uninitialized if someone cancels the copy early

RESOLVED FIXED

Status

()

Product:
Core
Component:
XPCOM
Importance:
--
trivial
Status:
RESOLVED FIXED
Reported:
10 years ago
Modified:
8 months ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

(Blocks: 1 bug, {coverity})

Version:
Trunk
Keywords:
coverity
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

fix reviewer
1.25 KB, patch
timeless
: review+
Details | Diff | Splinter Review
(Assignee)

Description

•
10 years ago 
Coverity is complaining that this code temporarily uses sourceCondition, sinkCondition uninitialized.

It's true, but the usage is local, and an optimizer could probably reach the same conclusion as my patch and do it automatically, but i'd rather not rely on that optimizer.
(Assignee)

Comment 1

•
10 years ago 
Created attachment 388544 [details] [diff] [review]
fix warning
Attachment #388544 - Flags: review?(doug.turner)

Updated

•
10 years ago
Attachment #388544 - Flags: review?(doug.turner) → review?(jduell.mcbugs)

Comment 2

•
10 years ago 
Comment on attachment 388544 [details] [diff] [review]
fix warning

This looks fine to me.   It's a really small change--do we even need sr?
Attachment #388544 - Flags: review?(jduell.mcbugs) → review+

Comment 3

•
10 years ago 
No SR needed.
 (Assignee)

Comment 4

•
10 years ago 
Created attachment 393756 [details] [diff] [review]
fix reviewer
Attachment #388544 - Attachment is obsolete: true
Attachment #393756 - Flags: review+
 (Assignee)

Comment 5

•
10 years ago 
http://hg.mozilla.org/mozilla-central/rev/df76fbd4b076
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.