<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
</div></div>Something doesn't look quite right - it seems that xmesa_init_display<br>
should be fixed instead.<br>
<br>
Currently it returns non-NULL when either of the following fail:<br>
 - driver.create_pipe_screen()<br>
 - CALLOC_STRUCT<br>
<br>
I would add an explicit check after those + goto err_path.<br>
The latter of which doing some proper teardown. Note: that there's a<br>
leak in there - one should walk, detach & free the new entry from the<br>
MesaExtInfo list.<br></blockquote><br></div><div class="gmail_quote">Rather than deal with the list cleanup after the fact, I can postpone the list insertion until the end when info is in a known good state.  That way when the error conditions are hit, the only cleanup that needs to be done is localized to screen and smapi.<br></div><div class="gmail_quote"><br></div><div class="gmail_quote">- Chuck<br></div><br></div></div>