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

Re: [libvirt] Confusing valgrind error in virDomainLookupByUUIDString or virReportErrorHelper



On 06/27/2011 10:32 AM, Richard W.M. Jones wrote:
> libvirt-0.9.2-2.fc16.x86_64

fc16 implies rawhide

> 
> Here's the error from valgrind:
> 
> ==24953== Conditional jump or move depends on uninitialised value(s)
> ==24953==    at 0x3DF68844BB: __GI___strcasecmp_l (strcmp.S:243)
> ==24953==    by 0x3DF68221E1: __gconv_open (gconv_open.c:70)
> ==24953==    by 0x3DF682F3E9: _nl_find_msg (dcigettext.c:974)
> ==24953==    by 0x3DF682FB84: __dcigettext (dcigettext.c:640)
> ==24953==    by 0x36BBE5A063: virReportErrorHelper (virterror.c:1215)

__dcigettext is a glibc function.  Sounds to me like valgrind is being
noisy about a recent glibc code change; this is likely a case of a
missing valgrind suppression file, and less likely an actual glibc bug.

> 
> Apparently __FUNCTION__ is deprecated[1].

Most likely a red herring.  We already '#define __FUNCTION__ __func__'
for any other C99 compliant compiler, so now it's just a global cleanup
to decide whether to consistently use gcc style or C99 style.

> That's as far as I've got for today.  Just letting people know ...

I'll see if I can reproduce it.  But since you were testing rawhide, my
guess is that it is a rawhide glibc/valgrind mismatch and not an actual
libvirt shortcoming.

-- 
Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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