<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - XWayland crashes during startup if output data is received"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=95337#c1">Comment # 1</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - XWayland crashes during startup if output data is received"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=95337">bug 95337</a>
              from <span class="vcard"><a class="email" href="mailto:fourdan@xfce.org" title="Olivier Fourdan <fourdan@xfce.org>"> <span class="fn">Olivier Fourdan</span></a>
</span></b>
        <pre>Created <span class=""><a href="attachment.cgi?id=123617" name="attach_123617" title="Possible fix">attachment 123617</a> <a href="attachment.cgi?id=123617&action=edit" title="Possible fix">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=95337&attachment=123617'>[review]</a>
Possible fix

(In reply to Mike Blumenkrantz from <a href="show_bug.cgi?id=95337#c0">comment #0</a>)
<span class="quote">> [...]
> ConnectionInfo is NULL, as InitInput is called before the connection data is
> initialized, meaning that anything trying to access it before this point
> will trigger a crash.</span >

Huh...

I am not sure, why we'd need to call the wl_display_roundtrip() in InitInput()?

This and initializing xwl_screen->expecting_event to 0 will be done again later
again in xwl_screen_init() where it seems more appropriate, and events can
safely be queued up meanwhile.

Does this patch help? I haven't detected any ill effect in my light initial
testing.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>