On 2011/06/21 23:49:07, groby wrote: > Please review Ugh, that was supposed to be target->saveCurrentDrawState(&fDrawState); ...
13 years, 5 months ago
(2011-06-22 01:28:52 UTC)
#2
On 2011/06/21 23:49:07, groby wrote:
> Please review
Ugh, that was supposed to be
target->saveCurrentDrawState(&fDrawState);
not
fDrawTarget->saveCurrentDrawState(&fDrawState);
inside the if.
Fixed at r1670.
Ah, thanks! So should I close my issue, or delete it? On 2011/06/22 01:28:52, brian2 ...
13 years, 5 months ago
(2011-06-22 01:34:06 UTC)
#3
Ah, thanks!
So should I close my issue, or delete it?
On 2011/06/22 01:28:52, brian2 wrote:
> On 2011/06/21 23:49:07, groby wrote:
> > Please review
>
> Ugh, that was supposed to be
>
> target->saveCurrentDrawState(&fDrawState);
> not
> fDrawTarget->saveCurrentDrawState(&fDrawState);
>
> inside the if.
>
> Fixed at r1670.
NP, sorry for the bug! :) I think closing it is fine. On 2011/06/22 01:34:06, ...
13 years, 5 months ago
(2011-06-22 01:39:15 UTC)
#4
NP, sorry for the bug! :) I think closing it is fine.
On 2011/06/22 01:34:06, groby wrote:
> Ah, thanks!
> So should I close my issue, or delete it?
>
>
> On 2011/06/22 01:28:52, brian2 wrote:
> > On 2011/06/21 23:49:07, groby wrote:
> > > Please review
> >
> > Ugh, that was supposed to be
> >
> > target->saveCurrentDrawState(&fDrawState);
> > not
> > fDrawTarget->saveCurrentDrawState(&fDrawState);
> >
> > inside the if.
> >
> > Fixed at r1670.
Issue 4639063: Fix possible NULL deref
(Closed)
Created 13 years, 5 months ago by groby
Modified 13 years, 5 months ago
Reviewers: vandebo (gmail), brian2
Base URL: http://skia.googlecode.com/svn/trunk/
Comments: 0