Re: [PATCH spice-gtk 04/14] channel: add SPICE_DISABLE_CHANNELS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 12, 2013 at 06:12:20PM +0200, Marc-André Lureau wrote:
> In glib:
> 
>   domains = g_getenv ("G_MESSAGES_DEBUG");
>   if (((log_level & INFO_LEVELS) == 0) ||
>       domains == NULL ||
>       (strcmp (domains, "all") != 0 && (!log_domain || !strstr
> (domains, log_domain))))
>     return;

Yup, found it too, which is not a great thing to do in this specific case
as external libraries can define their own log domain, so the strstr could
trigger false positives (eg gstreamer and gstr, a made-up string handling
library).

Christophe

Attachment: pgpIpVGjYf_Or.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]