<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 20, 2015 at 12:00 AM, Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, May 20, 2015 at 09:39:54AM +0300, Pekka Paalanen wrote:<br>
> On Wed, 20 May 2015 08:34:32 +0200<br>
> Hardening <<a href="mailto:rdp.effort@gmail.com">rdp.effort@gmail.com</a>> wrote:<br>
><br>
> > Le 20/05/2015 01:41, Bryce Harrington a écrit :<br>
> > > On Mon, May 18, 2015 at 11:14:16PM -0700, Dima Ryazanov wrote:<br>
> > >> When a compositor window is closed, remove the output instead of just exiting.<br>
> > >><br>
> > >> (The "if (!input->output)" checks are kind of ugly - but I couldn't find<br>
> > >> a better way to handle the output going away.)<br>
> > >><br>
> > >> Signed-off-by: Dima Ryazanov <<a href="mailto:dima@gmail.com">dima@gmail.com</a>><br>
> > > Reviewed-by: Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>><br>
> > ><br>
> > >> ---<br>
> > >>  src/compositor-wayland.c | 22 +++++++++++++++++++---<br>
> > >>  1 file changed, 19 insertions(+), 3 deletions(-)<br>
> > >><br>
> > [...]<br>
> ><br>
> > >><br>
> > >>                  if (frame_status(input->output->frame) & FRAME_STATUS_REPAINT)<br>
> > >>                          weston_output_schedule_repaint(&input->output->base);<br>
> > >> @@ -1521,7 +1537,7 @@ input_handle_keyboard_leave(void *data,<br>
> > >><br>
> > >>          focus = input->keyboard_focus;<br>
> > >>          if (!focus)<br>
> > >> -                return; /* This shouldn't happen */<br>
> > >> +                return;<br>
> ><br>
> > Just a remark, if it's something that shouldn't happen it's a failing<br>
> > assert not a silent return, isn't it ?<br>
><br>
> I'm assuming that after this patch, this actually can legally happen,<br>
> and in that case we should just ignore it, because we are getting<br>
> keyboard leave for something we already left by actively destroying it<br>
> to begin with. Right?<br>
<br>
</div></div>That's how I was understanding it.<br></blockquote><div><br></div><div>Yeah, exactly.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">Bryce<br>
</font></span></blockquote></div><br></div></div>