[PATCH] fix usage of unchecked returnval from hal_util_strdup_valid_utf8()

Kay Sievers kay.sievers at vrfy.org
Mon Sep 11 14:57:52 PDT 2006


On Mon, 2006-09-11 at 17:47 -0400, David Zeuthen wrote:
> On Mon, 2006-09-11 at 23:13 +0200, Danny Kukawka wrote:
> > Hi,
> > 
> > this patch fixes the usage of unchecked returnvalue from 
> > hal_util_strdup_valid_utf8(). The function returns NULL if there is no way to 
> > return a valid or a fixed validate string. A uncheck usage can cause this 
> > error:
> >     
> >       GLib-CRITICAL **: g_strlcpy: assertion `src != NULL' failed
> > 
> > Please review.
> 
> Looks good to me, thanks. Looks like udev ought to not provide data
> which is not UTF-8, it should probably be fixed in udev too. Or perhaps
> not, depends on udev and whether it just should return raw data. Kay?

Udev usually does not return any control characters, or non-utf-8, or
invalid utf-8 character sequences. If it does, let me know.

Thanks,
Kay



More information about the hal mailing list