[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