Closed Bug 576383 Opened 10 years ago Closed 10 years ago

AsyncChannel::MaybeHandleError case MsgProcessingError is missing break

Categories

(Core :: IPC, defect, minor)

defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity)

Attachments

(1 file)

372 AsyncChannel::MaybeHandleError(Result code, const char* channelName)
378     switch (code) {
388     case MsgProcessingError:
389         errorMsg = "Processing error: message was deserialized, but the 

390     case MsgRouteError:
391         errorMsg = "Route error: message sent to unknown actor ID";
392         break;
Blocks: 565844
Attached patch breakSplinter Review
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #455684 - Flags: review?(jones.chris.g)
Comment on attachment 455684 [details] [diff] [review]
break

Oops!  Kinda wish our static analysis box was still up (bug 522774).
Attachment #455684 - Flags: review?(jones.chris.g) → review+
http://hg.mozilla.org/mozilla-central/rev/1c2df0e3b4a9
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.