Open Bug 695498 Opened 8 years ago Updated 7 years ago

Cleanup WriteToDecoder a little

Categories

(Core :: ImageLib, defect)

x86
macOS
defect
Not set

Tracking

()

REOPENED
mozilla10

People

(Reporter: jrmuizel, Assigned: jrmuizel)

References

Details

Attachments

(1 file)

Removes the unobvious CONTAINER_ENSURE() which makes the fact that we only adjust
mBytesDecoded when we don't have success more obvious. This also takes out a check for mDecoder that is unneeded because of the kungFuDeathGrip.
Attachment #567865 - Flags: review?(bobbyholley+bmo)
Component: Graphics → ImageLib
QA Contact: thebes → imagelib
Comment on attachment 567865 [details] [diff] [review]
Cleanup WriteToDecoder a little

>+  if (status == NS_OK) {

if (NS_SUCCEEDED(status)) {

>+    // Keep track of the total number of bytes written over the lifetime of the
>+    // decoder

Can you add a period here?

r=bholley with that.
Attachment #567865 - Flags: review?(bobbyholley+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/7aa60d6408b3

[Please can you use the "assign bug to me" checkbox when attaching patches, as it makes the merge much quicker if the assignee has been filled out correctly - thanks :-) ]
Assignee: nobody → jmuizelaar
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
(In reply to Jeff Muizelaar [:jrmuizel] from comment #0)
> Created attachment 567865 [details] [diff] [review] [diff] [details] [review]
> Cleanup WriteToDecoder a little
> 
> Removes the unobvious CONTAINER_ENSURE() which makes the fact that we only
> adjust
> mBytesDecoded when we don't have success more obvious. This also takes out a
> check for mDecoder that is unneeded because of the kungFuDeathGrip.

WHOA. That is not the case. See bug 639303 comment 24. This needs to be fixed right now by backout, as it will reintroduce security bugs.
Reopening as fix was backed out. Although if the basis for doing this was wrong I guess this should be resolved another way...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Depends on: 700603
You need to log in before you can comment on or make changes to this bug.