<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div>it is needed to detect the return of qxl_uxa_init  in qxl_screen_init .<br><br><br><br><br></div><div></div><div id="divNeteaseMailCard"></div><div><br></div><pre><br>At 2013-11-06 21:59:58,"Jeremy White" <jwhite@codeweavers.com> wrote:
>Nice catch!
>
>On 11/06/2013 03:37 AM, bigclouds wrote:
>> hi, it allocate twice memory for qxl->uxa in function qxl_screen_init and qxl_uxa_init
>> -----------------
>> diff --git a/src/qxl_driver.c b/src/qxl_driver.c
>> index 91ba6c2..6be61e4 100644
>> --- a/src/qxl_driver.c
>> +++ b/src/qxl_driver.c
>> @@ -746,7 +746,9 @@ qxl_screen_init (SCREEN_INIT_ARGS_DECL)
>>       }
>>       qxl->uxa = uxa_driver_alloc ();
>> -
>> +    if (qxl->uxa == NULL)
>> +return FALSE;
>> +
>
>Wouldn't it be better to just delete this instance of the
>allocation and leave it all in qxl_uxa.c?
>
>Also, just a kibitz, but most open source projects require
>a full name on a submitted patch.
>
>Cheers,
>
>Jeremy
>_______________________________________________
>Spice-devel mailing list
>Spice-devel@lists.freedesktop.org
>http://lists.freedesktop.org/mailman/listinfo/spice-devel
</pre></div><br><br><span title="neteasefooter"><span id="netease_mail_footer"></span></span>