From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35233) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dL5sf-0000Z9-OK for qemu-devel@nongnu.org; Wed, 14 Jun 2017 06:50:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dL5sc-0003kh-2t for qemu-devel@nongnu.org; Wed, 14 Jun 2017 06:50:33 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43522) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dL5sb-0003kW-Pn for qemu-devel@nongnu.org; Wed, 14 Jun 2017 06:50:30 -0400 Date: Wed, 14 Jun 2017 11:50:23 +0100 From: "Daniel P. Berrange" Message-ID: <20170614105023.GG4370@redhat.com> Reply-To: "Daniel P. Berrange" References: <20170607175419.13558-1-berrange@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PULL v1 0/5] Merge sockets 2017/06/07 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers On Mon, Jun 12, 2017 at 02:14:19PM +0100, Peter Maydell wrote: > On 7 June 2017 at 18:54, Daniel P. Berrange wrote: > > The following changes since commit 0db1851becbefe3e50cfc03776fb1f75817376af: > > > > Merge remote-tracking branch 'remotes/vivier/tags/m68k-for-2.10-pull-request' into staging (2017-06-07 11:56:00 +0100) > > > > are available in the git repository at: > > > > git://github.com/berrange/qemu tags/pull-sockets-2017-06-07-1 > > > > for you to fetch changes up to 8dd2c27927b93aa352c43e8f0b9437f85d2f5f55: > > > > tests: add functional test validating ipv4/ipv6 address flag handling (2017-06-07 17:15:08 +0100) > > > > ---------------------------------------------------------------- > > Merge sockets 2017/06/07 v1 > > > > ---------------------------------------------------------------- > > Hi; I'm afraid this ran into 'make check' problems: > > On OSX and FreeBSD: > GTESTER check-qtest-i386 > ERROR:/Users/pm215/src/qemu-for-merges/tests/test-sockets-proto.c:825:void > test_listen(const void *): assertion failed: (data->ipv6 != 0) > GTester: last random seed: R02Sc21fa7784ab13d0fd3274d86c5a52367 > ** > ERROR:/Users/pm215/src/qemu-for-merges/tests/test-sockets-proto.c:825:void > test_listen(const void *): assertion failed: (data->ipv6 != 0) > GTester: last random seed: R02Sdf653a63e8d9a5d03bfadb49122f3e9a Ok, it appears that getaddrinfo() returns different results for "localhost" on FreeBSD/OSX, compared to Linux. Linux returns IPv4 first, then IPv6, while FreeBSD/OSX return IPv6 first then IPv4. The sockets code is working correctly, but the unit test is not seeing the expected results, since its data tables are expecting the Linux behaviour. Not sure how I'll fix this yet, perhaps I'll just #ifdef __linux__ as a temporary hack. > On the clang runtime undefined-behavior sanitizer: > GTESTER check-qtest-i386 > /home/petmay01/linaro/qemu-for-merges/qapi/qapi-visit-core.c:139:12: > runtime error: load of value 254, which is not a valid value for type > 'bool' > > (ditto for check-qtest-ppc64, x86_64 > GTESTER check-qtest-ppc64 > /home/petmay01/linaro/qemu-for-merges/qapi/qapi-visit-core.c:139:12: > runtime error: load of value 254, which is not a valid value for type > 'bool' [snip] > This usually means "forgot to initialize a bool". May or may not > be the same bug in both cases... Yes, this was a unit test bug using g_new instead of g_new0 Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|