On 2011/07/07 16:21:37, Peng wrote: lgtm So you fixed several use-after-free errors, correct? If so, ...
12 years, 11 months ago
(2011-07-07 16:34:34 UTC)
#2
On 2011/07/07 16:21:37, Peng wrote:
lgtm
So you fixed several use-after-free errors, correct? If so, should we merge it
to chromeos R13 branch? In other words, does the bug trigger a practical
problem, like ibus-daemon/chrome SEGV?
On 2011/07/07 16:34:34, Yusuke Sato wrote: > On 2011/07/07 16:21:37, Peng wrote: > > lgtm ...
12 years, 11 months ago
(2011-07-07 17:04:34 UTC)
#3
On 2011/07/07 16:34:34, Yusuke Sato wrote:
> On 2011/07/07 16:21:37, Peng wrote:
>
> lgtm
>
> So you fixed several use-after-free errors, correct? If so, should we merge it
> to chromeos R13 branch? In other words, does the bug trigger a practical
> problem, like ibus-daemon/chrome SEGV?
I think it is better to fix them in R13. I found sometime the hotkey does not
work without this fix. I am not sure if it is a same problem with
crosbug.com/15773 .
On 2011/07/07 17:04:34, Peng wrote: > On 2011/07/07 16:34:34, Yusuke Sato wrote: > > On ...
12 years, 11 months ago
(2011-07-07 17:43:16 UTC)
#4
On 2011/07/07 17:04:34, Peng wrote:
> On 2011/07/07 16:34:34, Yusuke Sato wrote:
> > On 2011/07/07 16:21:37, Peng wrote:
> >
> > lgtm
> >
> > So you fixed several use-after-free errors, correct? If so, should we merge
it
> > to chromeos R13 branch? In other words, does the bug trigger a practical
> > problem, like ibus-daemon/chrome SEGV?
>
> I think it is better to fix them in R13. I found sometime the hotkey does not
> work without this fix. I am not sure if it is a same problem with
> crosbug.com/15773 .
I just reproduced crosbug.com/15773 with this CL.
Issue 4667067: Fix several GVariant related issues. And remove a wrong unref.
(Closed)
Created 12 years, 11 months ago by Peng
Modified 12 years, 11 months ago
Reviewers: Yusuke Sato
Base URL: git@github.com:ibus/ibus.git@master
Comments: 0