[patch] Re: HAL and hotplugging

Dustin Spicuzza dustin at virtualroadside.com
Tue Mar 25 06:23:58 PDT 2008


You're right, sorry about that.

Sascha Hlusiak wrote:
> Hi,
>
> Am Mittwoch 19 März 2008 05:31:03 schrieb Dustin Spicuzza:
>   
>> Has anyone looked at this yet? It fixed the problems that were
>> previously mentioned, and I've been running it with my touchscreen since
>> Sunday, no issues yet.
>>     
> There is a memory allocation error in your patch for the legacy xkb option 
> parsing:
>
> tmp_val = get_prop_string(hal_ctx, udi, psi_key);
> if (tmp_val){
>   tmp_key = xalloc(strlen(tmp_val) + 5); /* add "xkb_" + NULL */
>
> Shouldn't that be 
>   tmp_key = xalloc(strlen(psi_key) - ( sizeof(LIBHAL_XKB_PROP_KEY) - 1) + 5);
>
> However, I think about testing and pushing this patch to master later on 
> today.
>
>
> - Sascha
>   


-- 
Innovation is just a problem away




More information about the xorg mailing list