[PATCH] add udi and some more checks to libhal

Rob Taylor rob.taylor at codethink.co.uk
Tue Dec 4 02:51:02 PST 2007


Danny Kukawka wrote:
> On Samstag, 24. November 2007, Danny Kukawka wrote:
>> Hi,
>>
>> this patch adds:
>> * a minimal check if a given UDI is valid (checks if the UDI starts
>> with '/org/freedesktop/Hal/devices/'). This should prevent possible
>> (application) crashes if a invalid UDI is send to HAL and reduce unneeded
>> dbus calls.
>> * some more parameter checks (for LibHalChangeSet, keys ...)
> 
> Any comments? Okay to commit?

Couple of things to fix:
 * s/UID/UDI.
 * Some unrelated whitespace changes, please do this in a separate commit.

Thanks,
Rob


> Danny
> _______________________________________________
> hal mailing list
> hal at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/hal


-- 
Rob Taylor, Codethink Ltd. -  http://codethink.co.uk


More information about the hal mailing list