[PATCH] add udi and some more checks to libhal

Danny Kukawka danny.kukawka at web.de
Thu Dec 6 01:30:53 PST 2007


On Dienstag, 4. Dezember 2007, Rob Taylor wrote:
> 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.

Commited,

Danny




More information about the hal mailing list