From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49835) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ampnW-0000mz-55 for qemu-devel@nongnu.org; Sun, 03 Apr 2016 17:43:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ampnS-00084A-WA for qemu-devel@nongnu.org; Sun, 03 Apr 2016 17:43:06 -0400 Received: from mail-lb0-x22a.google.com ([2a00:1450:4010:c04::22a]:34868) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ampnS-000844-Kl for qemu-devel@nongnu.org; Sun, 03 Apr 2016 17:43:02 -0400 Received: by mail-lb0-x22a.google.com with SMTP id bc4so133885883lbc.2 for ; Sun, 03 Apr 2016 14:43:02 -0700 (PDT) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\)) From: Liviu Ionescu In-Reply-To: Date: Mon, 4 Apr 2016 00:42:59 +0300 Content-Transfer-Encoding: quoted-printable Message-Id: <3F266835-BBA7-4F79-94A6-B23788577069@livius.net> References: <0F59BEF3-8BDD-434B-9EC4-3DECF9465A44@livius.net> <63AEB8A6-929B-48B2-B6D1-13FC80F8888F@livius.net> Subject: Re: [Qemu-devel] segmentation fault in object.c:type_initialize_interface() if interface not defined List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers > On 04 Apr 2016, at 00:30, Peter Maydell = wrote: >=20 > On 3 April 2016 at 18:56, Liviu Ionescu wrote: >> well, to summarise, I notified you that in certain conditions, >> due to an non-obvious dependency that does not break the build >> when not met, the existing code crashes at run time with a >> segmentation fault, and I suggested that an assert might help >> developers to save some debugging time. >=20 > Which I completely agree with. If you want to send a patch > to add the assert I'm happy to review it. I guess the original report was clear enough for an active commiter to = easily understand the problem and fix it, in significantly less = iterations that I would need to produce a compliant and acceptable patch = (reaching this point already took 9 messages). :-( regards, Liviu