From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028493389870.6183115327718; Wed, 5 Dec 2018 08:48:13 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 99E16C0495A8; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5C8E760BEE; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 129C93F609; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Glw2N028407 for ; Wed, 5 Dec 2018 11:47:58 -0500 Received: by smtp.corp.redhat.com (Postfix) id 35E135D96F; Wed, 5 Dec 2018 16:47:58 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 878694F9AB; Wed, 5 Dec 2018 16:47:57 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:42 +0100 Message-Id: <8a041d59ff75f6365ea2c4225444bb16e6f37d74.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 01/10] include: error: Add enum sentinel for virErrorNumber enum X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.31]); Wed, 05 Dec 2018 16:48:11 +0000 (UTC) Content-Type: text/plain; charset="utf-8" We do have one for the error domain but not for the error number itself. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- include/libvirt/virterror.h | 5 +++++ src/util/virerror.c | 1 + 2 files changed, 6 insertions(+) diff --git a/include/libvirt/virterror.h b/include/libvirt/virterror.h index 57aadb8d16..9867c05450 100644 --- a/include/libvirt/virterror.h +++ b/include/libvirt/virterror.h @@ -323,6 +323,11 @@ typedef enum { VIR_ERR_DEVICE_MISSING =3D 99, /* fail to find the desired devi= ce */ VIR_ERR_INVALID_NWFILTER_BINDING =3D 100, /* invalid nwfilter binding= */ VIR_ERR_NO_NWFILTER_BINDING =3D 101, /* no nwfilter binding */ + +# ifdef VIR_ENUM_SENTINELS + VIR_ERR_NUMBER_LAST +# endif + } virErrorNumber; /** diff --git a/src/util/virerror.c b/src/util/virerror.c index 683e51aa19..eca6ddf7d1 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -916,6 +916,7 @@ virErrorMsg(virErrorNumber error, const char *info) const char *errmsg =3D NULL; switch (error) { + case VIR_ERR_NUMBER_LAST: case VIR_ERR_OK: return NULL; case VIR_ERR_INTERNAL_ERROR: --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 154402848491214.03818966097549; Wed, 5 Dec 2018 08:48:04 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 706467F6B3; Wed, 5 Dec 2018 16:48:02 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 35C655D757; Wed, 5 Dec 2018 16:48:02 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 114293F7D5; Wed, 5 Dec 2018 16:48:01 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Glx7h028416 for ; Wed, 5 Dec 2018 11:47:59 -0500 Received: by smtp.corp.redhat.com (Postfix) id 32D155D96F; Wed, 5 Dec 2018 16:47:59 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 83F6A4F9AB; Wed, 5 Dec 2018 16:47:58 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:43 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 02/10] util: error: Fix error message strings to play well with additional info X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Wed, 05 Dec 2018 16:48:03 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Additional information for a string is always in form of a string or empty. Fix two offenders. One used %d as format modifier and second always expected a string. Both are thankfully unused currently. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- src/util/virerror.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/util/virerror.c b/src/util/virerror.c index eca6ddf7d1..5d515bde57 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -977,7 +977,10 @@ virErrorMsg(virErrorNumber error, const char *info) errmsg =3D _("POST operation failed"); break; case VIR_ERR_HTTP_ERROR: - errmsg =3D _("got unknown HTTP error code %d"); + if (info !=3D NULL) + errmsg =3D _("got unknown HTTP error code %s"); + else + errmsg =3D _("got unknown HTTP error code"); break; case VIR_ERR_UNKNOWN_HOST: if (info !=3D NULL) @@ -1004,7 +1007,10 @@ virErrorMsg(virErrorNumber error, const char *info) errmsg =3D _("could not connect to Xen Store %s"); break; case VIR_ERR_XEN_CALL: - errmsg =3D _("failed Xen syscall %s"); + if (info =3D=3D NULL) + errmsg =3D _("failed Xen syscall"); + else + errmsg =3D _("failed Xen syscall %s"); break; case VIR_ERR_OS_TYPE: if (info =3D=3D NULL) --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028484822303.61922105658937; Wed, 5 Dec 2018 08:48:04 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 75E1E307CDDA; Wed, 5 Dec 2018 16:48:02 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 3EF1A5D756; Wed, 5 Dec 2018 16:48:02 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id D85423F6E4; Wed, 5 Dec 2018 16:48:01 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm0Cj028426 for ; Wed, 5 Dec 2018 11:48:00 -0500 Received: by smtp.corp.redhat.com (Postfix) id 2EA5D5DA60; Wed, 5 Dec 2018 16:48:00 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 8094B5D96F; Wed, 5 Dec 2018 16:47:59 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:44 +0100 Message-Id: <13c03bc3f9c9dd464b4600cc582aff255d9ed67f.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 03/10] util: error: Add error message versions with info for some error codes X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.49]); Wed, 05 Dec 2018 16:48:03 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Few error codes were missing the version of the message with additional info. In case of the modified messages it's not very likely they'll ever report any additional data, but for the sake of consistency we'll provide them. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- src/util/virerror.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/src/util/virerror.c b/src/util/virerror.c index 5d515bde57..64be00dc75 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -926,7 +926,10 @@ virErrorMsg(virErrorNumber error, const char *info) errmsg =3D _("internal error"); break; case VIR_ERR_NO_MEMORY: - errmsg =3D _("out of memory"); + if (info =3D=3D NULL) + errmsg =3D _("out of memory"); + else + errmsg =3D _("out of memory: %s"); break; case VIR_ERR_NO_SUPPORT: if (info =3D=3D NULL) @@ -1019,7 +1022,10 @@ virErrorMsg(virErrorNumber error, const char *info) errmsg =3D _("unknown OS type %s"); break; case VIR_ERR_NO_KERNEL: - errmsg =3D _("missing kernel information"); + if (info =3D=3D NULL) + errmsg =3D _("missing kernel information"); + else + errmsg =3D _("missing kernel information: %s"); break; case VIR_ERR_NO_ROOT: if (info =3D=3D NULL) @@ -1472,7 +1478,10 @@ virErrorMsg(virErrorNumber error, const char *info) errmsg =3D _("XML document failed to validate against sche= ma: %s"); break; case VIR_ERR_MIGRATE_FINISH_OK: - errmsg =3D _("migration successfully aborted"); + if (info =3D=3D NULL) + errmsg =3D _("migration successfully aborted"); + else + errmsg =3D _("migration successfully aborted: %s"); break; case VIR_ERR_NO_SERVER: if (info =3D=3D NULL) --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028493132576.4962376036749; Wed, 5 Dec 2018 08:48:13 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id C1730308FBA4; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 8083F5DA9B; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 37E36181B9E7; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm10Z028436 for ; Wed, 5 Dec 2018 11:48:01 -0500 Received: by smtp.corp.redhat.com (Postfix) id 2A77B18BB8; Wed, 5 Dec 2018 16:48:01 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7B2DF5D96F; Wed, 5 Dec 2018 16:48:00 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:45 +0100 Message-Id: <9764301a70c249c6b674cb1ba9cb4ef99f660f04.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 04/10] util: error: Export virErrorMsg for use in testsuite X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.43]); Wed, 05 Dec 2018 16:48:11 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- src/Makefile.am | 1 + src/libvirt_private.syms | 1 + src/util/Makefile.inc.am | 1 + src/util/virerror.c | 5 ++++- src/util/virerrorpriv.h | 28 ++++++++++++++++++++++++++++ 5 files changed, 35 insertions(+), 1 deletion(-) create mode 100644 src/util/virerrorpriv.h diff --git a/src/Makefile.am b/src/Makefile.am index 33ff280d78..0bb3ff8fc8 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -951,6 +951,7 @@ libvirt_nss_la_SOURCES =3D \ util/vircommand.h \ util/virerror.c \ util/virerror.h \ + util/virerrorpriv.h \ util/virfile.c \ util/virfile.h \ util/virhash.c \ diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index fd63c9ca61..6184030d59 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -1752,6 +1752,7 @@ ebtablesRemoveForwardAllowIn; virDispatchError; virErrorCopyNew; virErrorInitialize; +virErrorMsg; virErrorPreserveLast; virErrorRestore; virErrorSetErrnoFromLastError; diff --git a/src/util/Makefile.inc.am b/src/util/Makefile.inc.am index cffbb357bc..4295babac3 100644 --- a/src/util/Makefile.inc.am +++ b/src/util/Makefile.inc.am @@ -50,6 +50,7 @@ UTIL_SOURCES =3D \ util/virendian.h \ util/virerror.c \ util/virerror.h \ + util/virerrorpriv.h \ util/virevent.c \ util/virevent.h \ util/vireventpoll.c \ diff --git a/src/util/virerror.c b/src/util/virerror.c index 64be00dc75..be23712a60 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -31,6 +31,9 @@ #include "virutil.h" #include "virstring.h" +#define __VIR_ERROR_ALLOW_INCLUDE_PRIV_H__ +#include "virerrorpriv.h" + VIR_LOG_INIT("util.error"); virThreadLocal virLastErr; @@ -910,7 +913,7 @@ void virRaiseErrorObject(const char *filename, * * Returns the constant string associated to @error */ -static const char * +const char * virErrorMsg(virErrorNumber error, const char *info) { const char *errmsg =3D NULL; diff --git a/src/util/virerrorpriv.h b/src/util/virerrorpriv.h new file mode 100644 index 0000000000..bc214393e6 --- /dev/null +++ b/src/util/virerrorpriv.h @@ -0,0 +1,28 @@ +/* + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library. If not, see + * . + */ + +#ifndef __VIR_ERROR_ALLOW_INCLUDE_PRIV_H__ +# error "virerrorpriv.h may only be included by virerror.c or its test sui= te" +#endif + +#ifndef __VIR_ERROR_PRIV_H__ +# define __VIR_ERROR_PRIV_H__ + +const char * +virErrorMsg(virErrorNumber error, + const char *info); + +#endif /* __VIR_ERROR_PRIV_H__ */ --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028497262624.1047616460734; Wed, 5 Dec 2018 08:48:17 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 773FD7F6BB; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 3EBDD5DA9B; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id ED02B3F60A; Wed, 5 Dec 2018 16:48:13 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm2CW028444 for ; Wed, 5 Dec 2018 11:48:02 -0500 Received: by smtp.corp.redhat.com (Postfix) id 258E818B95; Wed, 5 Dec 2018 16:48:02 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7698E5DA9B; Wed, 5 Dec 2018 16:48:01 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:46 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 05/10] tests: Add test for virErrorMsg message constraints X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Wed, 05 Dec 2018 16:48:15 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Make sure that we don't add any broken error message strings any more. This ensures that both the version with and without additional info is populated, the version without info does not have any formatting modifiers and the version with info has exactly one. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- tests/Makefile.am | 6 +++ tests/virerrortest.c | 101 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 107 insertions(+) create mode 100644 tests/virerrortest.c diff --git a/tests/Makefile.am b/tests/Makefile.am index d7ec7e3a6f..f2eef2293f 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -207,6 +207,7 @@ test_programs =3D virshtest sockettest \ virnetdevtest \ virtypedparamtest \ vshtabletest \ + virerrortest \ $(NULL) test_libraries =3D libshunload.la \ @@ -940,6 +941,11 @@ metadatatest_SOURCES =3D \ testutils.c testutils.h metadatatest_LDADD =3D $(LDADDS) $(LIBXML_LIBS) +virerrortest_SOURCES =3D \ + virerrortest.c \ + testutils.c testutils.h +virerrortest_LDADD =3D $(LDADDS) + vshtabletest_SOURCES =3D \ vshtabletest.c \ testutils.c testutils.h diff --git a/tests/virerrortest.c b/tests/virerrortest.c new file mode 100644 index 0000000000..0d0377bfa8 --- /dev/null +++ b/tests/virerrortest.c @@ -0,0 +1,101 @@ +/* + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; If not, see + * . + */ + +#include + +#include "testutils.h" + +#define __VIR_ERROR_ALLOW_INCLUDE_PRIV_H__ +#include "virerrorpriv.h" + +static int +virErrorTestMsgFormatInfoOne(const char *msg) +{ + bool found =3D false; + char *next; + int ret =3D 0; + + for (next =3D (char *)msg; (next =3D strchr(next, '%')); next++) { + if (next[1] !=3D 's') { + VIR_TEST_VERBOSE("\nerror message '%s' contains disallowed pri= ntf modifiers\n", msg); + ret =3D -1; + } else { + if (found) { + VIR_TEST_VERBOSE("\nerror message '%s' contains multiple %= %s modifiers\n", msg); + ret =3D -1; + } else { + found =3D true; + } + } + } + + if (!found) { + VIR_TEST_VERBOSE("\nerror message '%s' does not contain any %%s mo= difiers\n", msg); + ret =3D -1; + } + + return ret; +} + + +static int +virErrorTestMsgs(const void *opaque ATTRIBUTE_UNUSED) +{ + const char *err_noinfo; + const char *err_info; + size_t i; + int ret =3D 0; + + for (i =3D 1; i < VIR_ERR_NUMBER_LAST; i++) { + err_noinfo =3D virErrorMsg(i, NULL); + err_info =3D virErrorMsg(i, ""); + + if (!err_noinfo) { + VIR_TEST_VERBOSE("\nmissing string without info for error id %= zu\n", i); + ret =3D -1; + } + + if (!err_info) { + VIR_TEST_VERBOSE("\nmissing string with info for error id %zu\= n", i); + ret =3D -1; + } + + if (strchr(err_noinfo, '%')) { + VIR_TEST_VERBOSE("\nerror message id %zu contains formatting c= haracters: '%s'\n", + i, err_noinfo); + ret =3D -1; + } + + if (virErrorTestMsgFormatInfoOne(err_info) < 0) + ret =3D -1; + } + + return ret; +} + + +static int +mymain(void) +{ + int ret =3D 0; + + if (virTestRun("error message strings ", virErrorTestMsgs, NULL) < 0) + ret =3D -1; + + return ret =3D=3D 0 ? EXIT_SUCCESS : EXIT_FAILURE; +} + +VIR_TEST_MAIN(mymain) --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028495389101.6623571342941; Wed, 5 Dec 2018 08:48:15 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 6DA4730001D4; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 378D75D76B; Wed, 5 Dec 2018 16:48:10 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id DE62E181B9E5; Wed, 5 Dec 2018 16:48:09 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm3b0028451 for ; Wed, 5 Dec 2018 11:48:03 -0500 Received: by smtp.corp.redhat.com (Postfix) id 22F334F9AB; Wed, 5 Dec 2018 16:48:03 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 71C2F5DA60; Wed, 5 Dec 2018 16:48:02 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:47 +0100 Message-Id: <3a515a000cb5ea362dcbf0f1b4da06c8a2be79cb.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 06/10] util: error: Improve docs for virErrorMsg X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.44]); Wed, 05 Dec 2018 16:48:11 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Clarify how @info is used and how the returned values look like. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- src/util/virerror.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/util/virerror.c b/src/util/virerror.c index be23712a60..7444d671bb 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -906,12 +906,14 @@ void virRaiseErrorObject(const char *filename, /** * virErrorMsg: * @error: the virErrorNumber - * @info: usually the first parameter string + * @info: additional info string * - * Internal routine to get the message associated to an error raised - * from the library + * Internal routine to get the message associated to @error raised + * from the library. * - * Returns the constant string associated to @error + * Returns a *printf format string which describes @error. The returned st= ring + * contains exactly one '%s' modifier if @info is non-NULL, or no modifier= s at + * all if @info is NULL. If @error is invalid NULL is returned. */ const char * virErrorMsg(virErrorNumber error, const char *info) --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028497253825.0682961724794; Wed, 5 Dec 2018 08:48:17 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 45EAF89ACF; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 10BC05DA60; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id BF0B83F608; Wed, 5 Dec 2018 16:48:13 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm4ge028457 for ; Wed, 5 Dec 2018 11:48:04 -0500 Received: by smtp.corp.redhat.com (Postfix) id 1D0CD4F9AB; Wed, 5 Dec 2018 16:48:04 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6E7FB4F9D7; Wed, 5 Dec 2018 16:48:03 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:48 +0100 Message-Id: <3c38120022a5d476bf48e51837817d6918838409.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 07/10] DO NOT PUSH: Make sure that error messages are not moved around X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Wed, 05 Dec 2018 16:48:15 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Add a virerrortest case that makes sure that all error messages stay the same while refactoring. This patch will not be pushed upstream to avoid polluting the git history. --- tests/virerrormessages.txt | 101 +++++++++++++++++++++++++++++++++++++ tests/virerrortest.c | 25 +++++++++ 2 files changed, 126 insertions(+) create mode 100644 tests/virerrormessages.txt diff --git a/tests/virerrormessages.txt b/tests/virerrormessages.txt new file mode 100644 index 0000000000..7db66fa09b --- /dev/null +++ b/tests/virerrormessages.txt @@ -0,0 +1,101 @@ +1-internal error-internal error: %s +2-out of memory-out of memory: %s +3-this function is not supported by the connection driver-this function is= not supported by the connection driver: %s +4-unknown host-unknown host %s +5-no connection driver available-no connection driver available for %s +6-invalid connection pointer in-invalid connection pointer in %s +7-invalid domain pointer in-invalid domain pointer in %s +8-invalid argument-invalid argument: %s +9-operation failed-operation failed: %s +10-GET operation failed-GET operation failed: %s +11-POST operation failed-POST operation failed: %s +12-got unknown HTTP error code-got unknown HTTP error code %s +13-failed to serialize S-Expr-failed to serialize S-Expr: %s +14-could not use Xen hypervisor entry-could not use Xen hypervisor entry %s +15-failed Xen syscall-failed Xen syscall %s +16-unknown OS type-unknown OS type %s +17-missing kernel information-missing kernel information: %s +18-missing root device information-missing root device information in %s +19-missing source information for device-missing source information for de= vice %s +20-missing target information for device-missing target information for de= vice %s +21-missing name information-missing name information in %s +22-missing operating system information-missing operating system informati= on for %s +23-missing devices information-missing devices information for %s +24-could not connect to Xen Store-could not connect to Xen Store %s +25-too many drivers registered-too many drivers registered in %s +26-library call failed, possibly not supported-library call %s failed, pos= sibly not supported +27-XML description is invalid or not well formed-XML error: %s +28-this domain exists already-domain %s exists already +29-operation forbidden for read only access-operation forbidden: %s +30-failed to open configuration file for reading-failed to open %s for rea= ding +31-failed to read configuration file-failed to read configuration file %s +32-failed to parse configuration file-failed to parse configuration file %s +33-configuration file syntax error-configuration file syntax error: %s +34-failed to write configuration file-failed to write configuration file: = %s +35-parser error-%s +36-invalid network pointer in-invalid network pointer in %s +37-this network exists already-network %s exists already +38-system call error-%s +39-RPC error-%s +40-GNUTLS call error-%s +41-Failed to find the network-Failed to find the network: %s +42-Domain not found-Domain not found: %s +43-Network not found-Network not found: %s +44-invalid MAC address-invalid MAC address: %s +45-authentication failed-authentication failed: %s +46-invalid storage pool pointer in-invalid storage pool pointer in %s +47-invalid storage volume pointer in-invalid storage volume pointer in %s +48-Failed to find a storage driver-Failed to find a storage driver: %s +49-Storage pool not found-Storage pool not found: %s +50-Storage volume not found-Storage volume not found: %s +51-Failed to find a node driver-Failed to find a node driver: %s +52-invalid node device pointer-invalid node device pointer in %s +53-Node device not found-Node device not found: %s +54-Security model not found-Security model not found: %s +55-Requested operation is not valid-Requested operation is not valid: %s +56-Failed to find the interface-Failed to find the interface: %s +57-Interface not found-Interface not found: %s +58-invalid interface pointer in-invalid interface pointer in %s +59-multiple matching interfaces found-multiple matching interfaces found: = %s +60-Failed to start the nwfilter driver-Failed to start the nwfilter driver= : %s +61-Invalid network filter-Invalid network filter: %s +62-Network filter not found-Network filter not found: %s +63-Error while building firewall-Error while building firewall: %s +64-Failed to find a secret storage driver-Failed to find a secret storage = driver: %s +65-Invalid secret-Invalid secret: %s +66-Secret not found-Secret not found: %s +67-unsupported configuration-unsupported configuration: %s +68-Timed out during operation-Timed out during operation: %s +69-Failed to make domain persistent after migration-Failed to make domain = persistent after migration: %s +70-Hook script execution failed-Hook script execution failed: %s +71-Invalid snapshot-Invalid snapshot: %s +72-Domain snapshot not found-Domain snapshot not found: %s +73-invalid stream pointer-invalid stream pointer in %s +74-argument unsupported-argument unsupported: %s +75-Storage pool probe failed-Storage pool probe failed: %s +76-Storage pool already built-Storage pool already built: %s +77-revert requires force-revert requires force: %s +78-operation aborted-operation aborted: %s +79-authentication cancelled-authentication cancelled: %s +80-metadata not found-metadata not found: %s +81-Unsafe migration-Unsafe migration: %s +82-numerical overflow-numerical overflow: %s +83-block copy still active-block copy still active: %s +84-Operation not supported-Operation not supported: %s +85-SSH transport error-SSH transport error: %s +86-Guest agent is not responding-Guest agent is not responding: %s +87-resource busy-resource busy: %s +88-access denied-access denied: %s +89-error from service-error from service: %s +90-this storage volume exists already-storage volume %s exists already +91-the CPU is incompatible with host CPU-the CPU is incompatible with host= CPU: %s +92-XML document failed to validate against schema-XML document failed to v= alidate against schema: %s +93-migration successfully aborted-migration successfully aborted: %s +94-authentication unavailable-authentication unavailable: %s +95-Server not found-Server not found: %s +96-Client not found-Client not found: %s +97-guest agent replied with wrong id to guest-sync command-guest agent rep= lied with wrong id to guest-sync command: %s +98-libssh transport error-libssh transport error: %s +99-device not found-device not found: %s +100-Invalid network filter binding-Invalid network filter binding: %s +101-Network filter binding not found-Network filter binding not found: %s diff --git a/tests/virerrortest.c b/tests/virerrortest.c index 0d0377bfa8..e985ca743b 100644 --- a/tests/virerrortest.c +++ b/tests/virerrortest.c @@ -87,6 +87,29 @@ virErrorTestMsgs(const void *opaque ATTRIBUTE_UNUSED) } +static int +virErrorTestMsgsStable(const void *opaque ATTRIBUTE_UNUSED) +{ + virBuffer buf =3D VIR_BUFFER_INITIALIZER; + char *actual =3D NULL; + size_t i; + int ret =3D 0; + + for (i =3D 1; i < VIR_ERR_NUMBER_LAST; i++) { + virBufferAsprintf(&buf, "%zu-", i); + virBufferStrcat(&buf, virErrorMsg(i, NULL), "-", virErrorMsg(i, ""= ), "\n", NULL); + } + + actual =3D virBufferContentAndReset(&buf); + + if (virTestCompareToFile(actual, abs_srcdir "/virerrormessages.txt") <= 0) + ret =3D -1; + + VIR_FREE(actual); + return ret; +} + + static int mymain(void) { @@ -94,6 +117,8 @@ mymain(void) if (virTestRun("error message strings ", virErrorTestMsgs, NULL) < 0) ret =3D -1; + if (virTestRun("error message strings stability ", virErrorTestMsgsSta= ble, NULL) < 0) + ret =3D -1; return ret =3D=3D 0 ? EXIT_SUCCESS : EXIT_FAILURE; } --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028500754367.23883671981696; Wed, 5 Dec 2018 08:48:20 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 4DB07307CDC3; Wed, 5 Dec 2018 16:48:18 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 19DB35D780; Wed, 5 Dec 2018 16:48:18 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id BCAC93F615; Wed, 5 Dec 2018 16:48:17 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm5RN028478 for ; Wed, 5 Dec 2018 11:48:05 -0500 Received: by smtp.corp.redhat.com (Postfix) id 67EBA5DA60; Wed, 5 Dec 2018 16:48:05 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6B17A4F9AB; Wed, 5 Dec 2018 16:48:04 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:49 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 08/10] util: error: Use a more declarative approach in virErrorMsg X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.49]); Wed, 05 Dec 2018 16:48:19 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Use a macro to declare how the strings for individual error codes. This unifies the used condition and will allow simplifying the code further. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- src/libvirt_private.syms | 1 + src/util/virerror.c | 792 +++++++++------------------------------ src/util/virerrorpriv.h | 8 + 3 files changed, 188 insertions(+), 613 deletions(-) diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 6184030d59..775b33e151 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -1753,6 +1753,7 @@ virDispatchError; virErrorCopyNew; virErrorInitialize; virErrorMsg; +virErrorMsgStrings; virErrorPreserveLast; virErrorRestore; virErrorSetErrnoFromLastError; diff --git a/src/util/virerror.c b/src/util/virerror.c index 7444d671bb..d3cd06331f 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -903,6 +903,178 @@ void virRaiseErrorObject(const char *filename, } +const virErrorMsgTuple virErrorMsgStrings[VIR_ERR_NUMBER_LAST] =3D { + { VIR_ERR_OK, NULL, NULL }, + { VIR_ERR_INTERNAL_ERROR, "internal error", "internal error: %s" }, + { VIR_ERR_NO_MEMORY, "out of memory", "out of memory: %s" }, + { VIR_ERR_NO_SUPPORT, + "this function is not supported by the connection driver", + "this function is not supported by the connection driver: %s" }, + { VIR_ERR_UNKNOWN_HOST, "unknown host", "unknown host %s" }, + { VIR_ERR_NO_CONNECT, + "no connection driver available", + "no connection driver available for %s" }, + { VIR_ERR_INVALID_CONN, "invalid connection pointer in", "invalid conn= ection pointer in %s" }, + { VIR_ERR_INVALID_DOMAIN, "invalid domain pointer in", "invalid domain= pointer in %s" }, + { VIR_ERR_INVALID_ARG, "invalid argument", "invalid argument: %s" }, + { VIR_ERR_OPERATION_FAILED, "operation failed", "operation failed: %s"= }, + { VIR_ERR_GET_FAILED, "GET operation failed", "GET operation failed: %= s" }, + { VIR_ERR_POST_FAILED, "POST operation failed", "POST operation failed= : %s" }, + { VIR_ERR_HTTP_ERROR, "got unknown HTTP error code", "got unknown HTTP= error code %s" }, + { VIR_ERR_SEXPR_SERIAL, "failed to serialize S-Expr", "failed to seria= lize S-Expr: %s" }, + { VIR_ERR_NO_XEN, + "could not use Xen hypervisor entry", + "could not use Xen hypervisor entry %s" }, + { VIR_ERR_XEN_CALL, "failed Xen syscall", "failed Xen syscall %s" }, + { VIR_ERR_OS_TYPE, "unknown OS type", "unknown OS type %s" }, + { VIR_ERR_NO_KERNEL, "missing kernel information", "missing kernel inf= ormation: %s" }, + { VIR_ERR_NO_ROOT, "missing root device information", "missing root de= vice information in %s" }, + { VIR_ERR_NO_SOURCE, + "missing source information for device", + "missing source information for device %s" }, + { VIR_ERR_NO_TARGET, + "missing target information for device", + "missing target information for device %s" }, + { VIR_ERR_NO_NAME, "missing name information", "missing name informati= on in %s" }, + { VIR_ERR_NO_OS, + "missing operating system information", + "missing operating system information for %s" }, + { VIR_ERR_NO_DEVICE, "missing devices information", "missing devices i= nformation for %s" }, + { VIR_ERR_NO_XENSTORE, "could not connect to Xen Store", "could not co= nnect to Xen Store %s" }, + { VIR_ERR_DRIVER_FULL, "too many drivers registered", "too many driver= s registered in %s" }, + { VIR_ERR_CALL_FAILED, + "library call failed, possibly not supported", + "library call %s failed, possibly not supported" }, + { VIR_ERR_XML_ERROR, "XML description is invalid or not well formed", = "XML error: %s" }, + { VIR_ERR_DOM_EXIST, "this domain exists already", "domain %s exists a= lready" }, + { VIR_ERR_OPERATION_DENIED, + "operation forbidden for read only access", + "operation forbidden: %s" }, + { VIR_ERR_OPEN_FAILED, + "failed to open configuration file for reading", + "failed to open %s for reading" }, + { VIR_ERR_READ_FAILED, + "failed to read configuration file", + "failed to read configuration file %s" }, + { VIR_ERR_PARSE_FAILED, + "failed to parse configuration file", + "failed to parse configuration file %s" }, + { VIR_ERR_CONF_SYNTAX, + "configuration file syntax error", + "configuration file syntax error: %s" }, + { VIR_ERR_WRITE_FAILED, + "failed to write configuration file", + "failed to write configuration file: %s" }, + { VIR_ERR_XML_DETAIL, "parser error", "%s" }, + { VIR_ERR_INVALID_NETWORK, "invalid network pointer in", "invalid netw= ork pointer in %s" }, + { VIR_ERR_NETWORK_EXIST, "this network exists already", "network %s ex= ists already" }, + { VIR_ERR_SYSTEM_ERROR, "system call error", "%s" }, + { VIR_ERR_RPC, "RPC error", "%s" }, + { VIR_ERR_GNUTLS_ERROR, "GNUTLS call error", "%s" }, + { VIR_WAR_NO_NETWORK, "Failed to find the network", "Failed to find th= e network: %s" }, + { VIR_ERR_NO_DOMAIN, "Domain not found", "Domain not found: %s" }, + { VIR_ERR_NO_NETWORK, "Network not found", "Network not found: %s" }, + { VIR_ERR_INVALID_MAC, "invalid MAC address", "invalid MAC address: %s= " }, + { VIR_ERR_AUTH_FAILED, "authentication failed", "authentication failed= : %s" }, + { VIR_ERR_INVALID_STORAGE_POOL, + "invalid storage pool pointer in", + "invalid storage pool pointer in %s" }, + { VIR_ERR_INVALID_STORAGE_VOL, + "invalid storage volume pointer in", + "invalid storage volume pointer in %s" }, + { VIR_WAR_NO_STORAGE, + "Failed to find a storage driver", + "Failed to find a storage driver: %s" }, + { VIR_ERR_NO_STORAGE_POOL, "Storage pool not found", "Storage pool not= found: %s" }, + { VIR_ERR_NO_STORAGE_VOL, "Storage volume not found", "Storage volume = not found: %s" }, + { VIR_WAR_NO_NODE, "Failed to find a node driver", "Failed to find a n= ode driver: %s" }, + { VIR_ERR_INVALID_NODE_DEVICE, + "invalid node device pointer", + "invalid node device pointer in %s" }, + { VIR_ERR_NO_NODE_DEVICE, "Node device not found", "Node device not fo= und: %s" }, + { VIR_ERR_NO_SECURITY_MODEL, "Security model not found", "Security mod= el not found: %s" }, + { VIR_ERR_OPERATION_INVALID, + "Requested operation is not valid", + "Requested operation is not valid: %s" }, + { VIR_WAR_NO_INTERFACE, "Failed to find the interface", "Failed to fin= d the interface: %s" }, + { VIR_ERR_NO_INTERFACE, "Interface not found", "Interface not found: %= s" }, + { VIR_ERR_INVALID_INTERFACE, + "invalid interface pointer in", + "invalid interface pointer in %s" }, + { VIR_ERR_MULTIPLE_INTERFACES, + "multiple matching interfaces found", + "multiple matching interfaces found: %s" }, + { VIR_WAR_NO_NWFILTER, + "Failed to start the nwfilter driver", + "Failed to start the nwfilter driver: %s" }, + { VIR_ERR_INVALID_NWFILTER, "Invalid network filter", "Invalid network= filter: %s" }, + { VIR_ERR_NO_NWFILTER, "Network filter not found", "Network filter not= found: %s" }, + { VIR_ERR_BUILD_FIREWALL, + "Error while building firewall", + "Error while building firewall: %s" }, + { VIR_WAR_NO_SECRET, + "Failed to find a secret storage driver", + "Failed to find a secret storage driver: %s" }, + { VIR_ERR_INVALID_SECRET, "Invalid secret", "Invalid secret: %s" }, + { VIR_ERR_NO_SECRET, "Secret not found", "Secret not found: %s" }, + { VIR_ERR_CONFIG_UNSUPPORTED, "unsupported configuration", "unsupporte= d configuration: %s" }, + { VIR_ERR_OPERATION_TIMEOUT, "Timed out during operation", "Timed out = during operation: %s" }, + { VIR_ERR_MIGRATE_PERSIST_FAILED, + "Failed to make domain persistent after migration", + "Failed to make domain persistent after migration: %s" }, + { VIR_ERR_HOOK_SCRIPT_FAILED, + "Hook script execution failed", + "Hook script execution failed: %s" }, + { VIR_ERR_INVALID_DOMAIN_SNAPSHOT, "Invalid snapshot", "Invalid snapsh= ot: %s" }, + { VIR_ERR_NO_DOMAIN_SNAPSHOT, "Domain snapshot not found", "Domain sna= pshot not found: %s" }, + { VIR_ERR_INVALID_STREAM, "invalid stream pointer", "invalid stream po= inter in %s" }, + { VIR_ERR_ARGUMENT_UNSUPPORTED, "argument unsupported", "argument unsu= pported: %s" }, + { VIR_ERR_STORAGE_PROBE_FAILED, "Storage pool probe failed", "Storage = pool probe failed: %s" }, + { VIR_ERR_STORAGE_POOL_BUILT, "Storage pool already built", "Storage p= ool already built: %s" }, + { VIR_ERR_SNAPSHOT_REVERT_RISKY, "revert requires force", "revert requ= ires force: %s" }, + { VIR_ERR_OPERATION_ABORTED, "operation aborted", "operation aborted: = %s" }, + { VIR_ERR_AUTH_CANCELLED, "authentication cancelled", "authentication = cancelled: %s" }, + { VIR_ERR_NO_DOMAIN_METADATA, "metadata not found", "metadata not foun= d: %s" }, + { VIR_ERR_MIGRATE_UNSAFE, "Unsafe migration", "Unsafe migration: %s" }, + { VIR_ERR_OVERFLOW, "numerical overflow", "numerical overflow: %s" }, + { VIR_ERR_BLOCK_COPY_ACTIVE, "block copy still active", "block copy st= ill active: %s" }, + { VIR_ERR_OPERATION_UNSUPPORTED, "Operation not supported", "Operation= not supported: %s" }, + { VIR_ERR_SSH, "SSH transport error", "SSH transport error: %s" }, + { VIR_ERR_AGENT_UNRESPONSIVE, + "Guest agent is not responding", + "Guest agent is not responding: %s" }, + { VIR_ERR_RESOURCE_BUSY, "resource busy", "resource busy: %s" }, + { VIR_ERR_ACCESS_DENIED, "access denied", "access denied: %s" }, + { VIR_ERR_DBUS_SERVICE, "error from service", "error from service: %s"= }, + { VIR_ERR_STORAGE_VOL_EXIST, + "this storage volume exists already", + "storage volume %s exists already" }, + { VIR_ERR_CPU_INCOMPATIBLE, + "the CPU is incompatible with host CPU", + "the CPU is incompatible with host CPU: %s" }, + { VIR_ERR_XML_INVALID_SCHEMA, + "XML document failed to validate against schema", + "XML document failed to validate against schema: %s" }, + { VIR_ERR_MIGRATE_FINISH_OK, + "migration successfully aborted", + "migration successfully aborted: %s" }, + { VIR_ERR_AUTH_UNAVAILABLE, "authentication unavailable", "authenticat= ion unavailable: %s" }, + { VIR_ERR_NO_SERVER, "Server not found", "Server not found: %s" }, + { VIR_ERR_NO_CLIENT, "Client not found", "Client not found: %s" }, + { VIR_ERR_AGENT_UNSYNCED, + "guest agent replied with wrong id to guest-sync command", + "guest agent replied with wrong id to guest-sync command: %s" }, + { VIR_ERR_LIBSSH, "libssh transport error", "libssh transport error: %= s" }, + { VIR_ERR_DEVICE_MISSING, "device not found", "device not found: %s" }, + { VIR_ERR_INVALID_NWFILTER_BINDING, + "Invalid network filter binding", + "Invalid network filter binding: %s" }, + { VIR_ERR_NO_NWFILTER_BINDING, + "Network filter binding not found", + "Network filter binding not found: %s" }, +}; + + /** * virErrorMsg: * @error: the virErrorNumber @@ -918,622 +1090,16 @@ void virRaiseErrorObject(const char *filename, const char * virErrorMsg(virErrorNumber error, const char *info) { - const char *errmsg =3D NULL; + if (error >=3D VIR_ERR_NUMBER_LAST) + return NULL; - switch (error) { - case VIR_ERR_NUMBER_LAST: - case VIR_ERR_OK: - return NULL; - case VIR_ERR_INTERNAL_ERROR: - if (info !=3D NULL) - errmsg =3D _("internal error: %s"); - else - errmsg =3D _("internal error"); - break; - case VIR_ERR_NO_MEMORY: - if (info =3D=3D NULL) - errmsg =3D _("out of memory"); - else - errmsg =3D _("out of memory: %s"); - break; - case VIR_ERR_NO_SUPPORT: - if (info =3D=3D NULL) - errmsg =3D _("this function is not supported by the connec= tion driver"); - else - errmsg =3D _("this function is not supported by the connec= tion driver: %s"); - break; - case VIR_ERR_NO_CONNECT: - if (info =3D=3D NULL) - errmsg =3D _("no connection driver available"); - else - errmsg =3D _("no connection driver available for %s"); - break; - case VIR_ERR_INVALID_CONN: - if (info =3D=3D NULL) - errmsg =3D _("invalid connection pointer in"); - else - errmsg =3D _("invalid connection pointer in %s"); - break; - case VIR_ERR_INVALID_DOMAIN: - if (info =3D=3D NULL) - errmsg =3D _("invalid domain pointer in"); - else - errmsg =3D _("invalid domain pointer in %s"); - break; - case VIR_ERR_INVALID_ARG: - if (info =3D=3D NULL) - errmsg =3D _("invalid argument"); - else - errmsg =3D _("invalid argument: %s"); - break; - case VIR_ERR_OPERATION_FAILED: - if (info !=3D NULL) - errmsg =3D _("operation failed: %s"); - else - errmsg =3D _("operation failed"); - break; - case VIR_ERR_GET_FAILED: - if (info !=3D NULL) - errmsg =3D _("GET operation failed: %s"); - else - errmsg =3D _("GET operation failed"); - break; - case VIR_ERR_POST_FAILED: - if (info !=3D NULL) - errmsg =3D _("POST operation failed: %s"); - else - errmsg =3D _("POST operation failed"); - break; - case VIR_ERR_HTTP_ERROR: - if (info !=3D NULL) - errmsg =3D _("got unknown HTTP error code %s"); - else - errmsg =3D _("got unknown HTTP error code"); - break; - case VIR_ERR_UNKNOWN_HOST: - if (info !=3D NULL) - errmsg =3D _("unknown host %s"); - else - errmsg =3D _("unknown host"); - break; - case VIR_ERR_SEXPR_SERIAL: - if (info !=3D NULL) - errmsg =3D _("failed to serialize S-Expr: %s"); - else - errmsg =3D _("failed to serialize S-Expr"); - break; - case VIR_ERR_NO_XEN: - if (info =3D=3D NULL) - errmsg =3D _("could not use Xen hypervisor entry"); - else - errmsg =3D _("could not use Xen hypervisor entry %s"); - break; - case VIR_ERR_NO_XENSTORE: - if (info =3D=3D NULL) - errmsg =3D _("could not connect to Xen Store"); - else - errmsg =3D _("could not connect to Xen Store %s"); - break; - case VIR_ERR_XEN_CALL: - if (info =3D=3D NULL) - errmsg =3D _("failed Xen syscall"); - else - errmsg =3D _("failed Xen syscall %s"); - break; - case VIR_ERR_OS_TYPE: - if (info =3D=3D NULL) - errmsg =3D _("unknown OS type"); - else - errmsg =3D _("unknown OS type %s"); - break; - case VIR_ERR_NO_KERNEL: - if (info =3D=3D NULL) - errmsg =3D _("missing kernel information"); - else - errmsg =3D _("missing kernel information: %s"); - break; - case VIR_ERR_NO_ROOT: - if (info =3D=3D NULL) - errmsg =3D _("missing root device information"); - else - errmsg =3D _("missing root device information in %s"); - break; - case VIR_ERR_NO_SOURCE: - if (info =3D=3D NULL) - errmsg =3D _("missing source information for device"); - else - errmsg =3D _("missing source information for device %s"); - break; - case VIR_ERR_NO_TARGET: - if (info =3D=3D NULL) - errmsg =3D _("missing target information for device"); - else - errmsg =3D _("missing target information for device %s"); - break; - case VIR_ERR_NO_NAME: - if (info =3D=3D NULL) - errmsg =3D _("missing name information"); - else - errmsg =3D _("missing name information in %s"); - break; - case VIR_ERR_NO_OS: - if (info =3D=3D NULL) - errmsg =3D _("missing operating system information"); - else - errmsg =3D _("missing operating system information for %s"= ); - break; - case VIR_ERR_NO_DEVICE: - if (info =3D=3D NULL) - errmsg =3D _("missing devices information"); - else - errmsg =3D _("missing devices information for %s"); - break; - case VIR_ERR_DRIVER_FULL: - if (info =3D=3D NULL) - errmsg =3D _("too many drivers registered"); - else - errmsg =3D _("too many drivers registered in %s"); - break; - case VIR_ERR_CALL_FAILED: /* DEPRECATED, use VIR_ERR_NO_SUPPORT */ - if (info =3D=3D NULL) - errmsg =3D _("library call failed, possibly not supported"= ); - else - errmsg =3D _("library call %s failed, possibly not support= ed"); - break; - case VIR_ERR_XML_ERROR: - if (info =3D=3D NULL) - errmsg =3D _("XML description is invalid or not well forme= d"); - else - errmsg =3D _("XML error: %s"); - break; - case VIR_ERR_DOM_EXIST: - if (info =3D=3D NULL) - errmsg =3D _("this domain exists already"); - else - errmsg =3D _("domain %s exists already"); - break; - case VIR_ERR_OPERATION_DENIED: - if (info =3D=3D NULL) - errmsg =3D _("operation forbidden for read only access"); - else - errmsg =3D _("operation forbidden: %s"); - break; - case VIR_ERR_OPEN_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("failed to open configuration file for readin= g"); - else - errmsg =3D _("failed to open %s for reading"); - break; - case VIR_ERR_READ_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("failed to read configuration file"); - else - errmsg =3D _("failed to read configuration file %s"); - break; - case VIR_ERR_PARSE_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("failed to parse configuration file"); - else - errmsg =3D _("failed to parse configuration file %s"); - break; - case VIR_ERR_CONF_SYNTAX: - if (info =3D=3D NULL) - errmsg =3D _("configuration file syntax error"); - else - errmsg =3D _("configuration file syntax error: %s"); - break; - case VIR_ERR_WRITE_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("failed to write configuration file"); - else - errmsg =3D _("failed to write configuration file: %s"); - break; - case VIR_ERR_XML_DETAIL: - if (info =3D=3D NULL) - errmsg =3D _("parser error"); - else - errmsg =3D "%s"; - break; - case VIR_ERR_INVALID_NETWORK: - if (info =3D=3D NULL) - errmsg =3D _("invalid network pointer in"); - else - errmsg =3D _("invalid network pointer in %s"); - break; - case VIR_ERR_NETWORK_EXIST: - if (info =3D=3D NULL) - errmsg =3D _("this network exists already"); - else - errmsg =3D _("network %s exists already"); - break; - case VIR_ERR_SYSTEM_ERROR: - if (info =3D=3D NULL) - errmsg =3D _("system call error"); - else - errmsg =3D "%s"; - break; - case VIR_ERR_RPC: - if (info =3D=3D NULL) - errmsg =3D _("RPC error"); - else - errmsg =3D "%s"; - break; - case VIR_ERR_GNUTLS_ERROR: - if (info =3D=3D NULL) - errmsg =3D _("GNUTLS call error"); - else - errmsg =3D "%s"; - break; - case VIR_WAR_NO_NETWORK: - if (info =3D=3D NULL) - errmsg =3D _("Failed to find the network"); - else - errmsg =3D _("Failed to find the network: %s"); - break; - case VIR_ERR_NO_DOMAIN: - if (info =3D=3D NULL) - errmsg =3D _("Domain not found"); - else - errmsg =3D _("Domain not found: %s"); - break; - case VIR_ERR_NO_NETWORK: - if (info =3D=3D NULL) - errmsg =3D _("Network not found"); - else - errmsg =3D _("Network not found: %s"); - break; - case VIR_ERR_INVALID_MAC: - if (info =3D=3D NULL) - errmsg =3D _("invalid MAC address"); - else - errmsg =3D _("invalid MAC address: %s"); - break; - case VIR_ERR_AUTH_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("authentication failed"); - else - errmsg =3D _("authentication failed: %s"); - break; - case VIR_ERR_AUTH_CANCELLED: - if (info =3D=3D NULL) - errmsg =3D _("authentication cancelled"); - else - errmsg =3D _("authentication cancelled: %s"); - break; - case VIR_ERR_AUTH_UNAVAILABLE: - if (info =3D=3D NULL) - errmsg =3D _("authentication unavailable"); - else - errmsg =3D _("authentication unavailable: %s"); - break; - case VIR_ERR_NO_STORAGE_POOL: - if (info =3D=3D NULL) - errmsg =3D _("Storage pool not found"); - else - errmsg =3D _("Storage pool not found: %s"); - break; - case VIR_ERR_NO_STORAGE_VOL: - if (info =3D=3D NULL) - errmsg =3D _("Storage volume not found"); - else - errmsg =3D _("Storage volume not found: %s"); - break; - case VIR_ERR_STORAGE_VOL_EXIST: - if (info =3D=3D NULL) - errmsg =3D _("this storage volume exists already"); - else - errmsg =3D _("storage volume %s exists already"); - break; - case VIR_ERR_STORAGE_PROBE_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("Storage pool probe failed"); - else - errmsg =3D _("Storage pool probe failed: %s"); - break; - case VIR_ERR_STORAGE_POOL_BUILT: - if (info =3D=3D NULL) - errmsg =3D _("Storage pool already built"); - else - errmsg =3D _("Storage pool already built: %s"); - break; - case VIR_ERR_INVALID_STORAGE_POOL: - if (info =3D=3D NULL) - errmsg =3D _("invalid storage pool pointer in"); - else - errmsg =3D _("invalid storage pool pointer in %s"); - break; - case VIR_ERR_INVALID_STORAGE_VOL: - if (info =3D=3D NULL) - errmsg =3D _("invalid storage volume pointer in"); - else - errmsg =3D _("invalid storage volume pointer in %s"); - break; - case VIR_WAR_NO_STORAGE: - if (info =3D=3D NULL) - errmsg =3D _("Failed to find a storage driver"); - else - errmsg =3D _("Failed to find a storage driver: %s"); - break; - case VIR_WAR_NO_NODE: - if (info =3D=3D NULL) - errmsg =3D _("Failed to find a node driver"); - else - errmsg =3D _("Failed to find a node driver: %s"); - break; - case VIR_ERR_INVALID_NODE_DEVICE: - if (info =3D=3D NULL) - errmsg =3D _("invalid node device pointer"); - else - errmsg =3D _("invalid node device pointer in %s"); - break; - case VIR_ERR_NO_NODE_DEVICE: - if (info =3D=3D NULL) - errmsg =3D _("Node device not found"); - else - errmsg =3D _("Node device not found: %s"); - break; - case VIR_ERR_NO_SECURITY_MODEL: - if (info =3D=3D NULL) - errmsg =3D _("Security model not found"); - else - errmsg =3D _("Security model not found: %s"); - break; - case VIR_ERR_OPERATION_INVALID: - if (info =3D=3D NULL) - errmsg =3D _("Requested operation is not valid"); - else - errmsg =3D _("Requested operation is not valid: %s"); - break; - case VIR_WAR_NO_INTERFACE: - if (info =3D=3D NULL) - errmsg =3D _("Failed to find the interface"); - else - errmsg =3D _("Failed to find the interface: %s"); - break; - case VIR_ERR_NO_INTERFACE: - if (info =3D=3D NULL) - errmsg =3D _("Interface not found"); - else - errmsg =3D _("Interface not found: %s"); - break; - case VIR_ERR_INVALID_INTERFACE: - if (info =3D=3D NULL) - errmsg =3D _("invalid interface pointer in"); - else - errmsg =3D _("invalid interface pointer in %s"); - break; - case VIR_ERR_MULTIPLE_INTERFACES: - if (info =3D=3D NULL) - errmsg =3D _("multiple matching interfaces found"); - else - errmsg =3D _("multiple matching interfaces found: %s"); - break; - case VIR_WAR_NO_SECRET: - if (info =3D=3D NULL) - errmsg =3D _("Failed to find a secret storage driver"); - else - errmsg =3D _("Failed to find a secret storage driver: %s"); - break; - case VIR_ERR_INVALID_SECRET: - if (info =3D=3D NULL) - errmsg =3D _("Invalid secret"); - else - errmsg =3D _("Invalid secret: %s"); - break; - case VIR_ERR_NO_SECRET: - if (info =3D=3D NULL) - errmsg =3D _("Secret not found"); - else - errmsg =3D _("Secret not found: %s"); - break; - case VIR_WAR_NO_NWFILTER: - if (info =3D=3D NULL) - errmsg =3D _("Failed to start the nwfilter driver"); - else - errmsg =3D _("Failed to start the nwfilter driver: %s"); - break; - case VIR_ERR_INVALID_NWFILTER: - if (info =3D=3D NULL) - errmsg =3D _("Invalid network filter"); - else - errmsg =3D _("Invalid network filter: %s"); - break; - case VIR_ERR_NO_NWFILTER: - if (info =3D=3D NULL) - errmsg =3D _("Network filter not found"); - else - errmsg =3D _("Network filter not found: %s"); - break; - case VIR_ERR_BUILD_FIREWALL: - if (info =3D=3D NULL) - errmsg =3D _("Error while building firewall"); - else - errmsg =3D _("Error while building firewall: %s"); - break; - case VIR_ERR_CONFIG_UNSUPPORTED: - if (info =3D=3D NULL) - errmsg =3D _("unsupported configuration"); - else - errmsg =3D _("unsupported configuration: %s"); - break; - case VIR_ERR_OPERATION_TIMEOUT: - if (info =3D=3D NULL) - errmsg =3D _("Timed out during operation"); - else - errmsg =3D _("Timed out during operation: %s"); - break; - case VIR_ERR_MIGRATE_PERSIST_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("Failed to make domain persistent after migra= tion"); - else - errmsg =3D _("Failed to make domain persistent after migra= tion: %s"); - break; - case VIR_ERR_HOOK_SCRIPT_FAILED: - if (info =3D=3D NULL) - errmsg =3D _("Hook script execution failed"); - else - errmsg =3D _("Hook script execution failed: %s"); - break; - case VIR_ERR_INVALID_DOMAIN_SNAPSHOT: - if (info =3D=3D NULL) - errmsg =3D _("Invalid snapshot"); - else - errmsg =3D _("Invalid snapshot: %s"); - break; - case VIR_ERR_NO_DOMAIN_SNAPSHOT: - if (info =3D=3D NULL) - errmsg =3D _("Domain snapshot not found"); - else - errmsg =3D _("Domain snapshot not found: %s"); - break; - case VIR_ERR_INVALID_STREAM: - if (info =3D=3D NULL) - errmsg =3D _("invalid stream pointer"); - else - errmsg =3D _("invalid stream pointer in %s"); - break; - case VIR_ERR_ARGUMENT_UNSUPPORTED: - if (info =3D=3D NULL) - errmsg =3D _("argument unsupported"); - else - errmsg =3D _("argument unsupported: %s"); - break; - case VIR_ERR_SNAPSHOT_REVERT_RISKY: - if (info =3D=3D NULL) - errmsg =3D _("revert requires force"); - else - errmsg =3D _("revert requires force: %s"); - break; - case VIR_ERR_OPERATION_ABORTED: - if (info =3D=3D NULL) - errmsg =3D _("operation aborted"); - else - errmsg =3D _("operation aborted: %s"); - break; - case VIR_ERR_NO_DOMAIN_METADATA: - if (info =3D=3D NULL) - errmsg =3D _("metadata not found"); - else - errmsg =3D _("metadata not found: %s"); - break; - case VIR_ERR_MIGRATE_UNSAFE: - if (!info) - errmsg =3D _("Unsafe migration"); - else - errmsg =3D _("Unsafe migration: %s"); - break; - case VIR_ERR_OVERFLOW: - if (!info) - errmsg =3D _("numerical overflow"); - else - errmsg =3D _("numerical overflow: %s"); - break; - case VIR_ERR_BLOCK_COPY_ACTIVE: - if (!info) - errmsg =3D _("block copy still active"); - else - errmsg =3D _("block copy still active: %s"); - break; - case VIR_ERR_OPERATION_UNSUPPORTED: - if (!info) - errmsg =3D _("Operation not supported"); - else - errmsg =3D _("Operation not supported: %s"); - break; - case VIR_ERR_SSH: - if (info =3D=3D NULL) - errmsg =3D _("SSH transport error"); - else - errmsg =3D _("SSH transport error: %s"); - break; - case VIR_ERR_AGENT_UNRESPONSIVE: - if (info =3D=3D NULL) - errmsg =3D _("Guest agent is not responding"); - else - errmsg =3D _("Guest agent is not responding: %s"); - break; - case VIR_ERR_RESOURCE_BUSY: - if (info =3D=3D NULL) - errmsg =3D _("resource busy"); - else - errmsg =3D _("resource busy: %s"); - break; - case VIR_ERR_ACCESS_DENIED: - if (info =3D=3D NULL) - errmsg =3D _("access denied"); - else - errmsg =3D _("access denied: %s"); - break; - case VIR_ERR_DBUS_SERVICE: - if (info =3D=3D NULL) - errmsg =3D _("error from service"); - else - errmsg =3D _("error from service: %s"); - break; - case VIR_ERR_CPU_INCOMPATIBLE: - if (info =3D=3D NULL) - errmsg =3D _("the CPU is incompatible with host CPU"); - else - errmsg =3D _("the CPU is incompatible with host CPU: %s"); - break; - case VIR_ERR_XML_INVALID_SCHEMA: - if (info =3D=3D NULL) - errmsg =3D _("XML document failed to validate against sche= ma"); - else - errmsg =3D _("XML document failed to validate against sche= ma: %s"); - break; - case VIR_ERR_MIGRATE_FINISH_OK: - if (info =3D=3D NULL) - errmsg =3D _("migration successfully aborted"); - else - errmsg =3D _("migration successfully aborted: %s"); - break; - case VIR_ERR_NO_SERVER: - if (info =3D=3D NULL) - errmsg =3D _("Server not found"); - else - errmsg =3D _("Server not found: %s"); - break; - case VIR_ERR_NO_CLIENT: - if (info =3D=3D NULL) - errmsg =3D _("Client not found"); - else - errmsg =3D _("Client not found: %s"); - break; - case VIR_ERR_AGENT_UNSYNCED: /* DEPRECATED */ - if (info =3D=3D NULL) - errmsg =3D _("guest agent replied with wrong id to guest-s= ync command"); - else - errmsg =3D _("guest agent replied with wrong id to guest-s= ync command: %s"); - break; - case VIR_ERR_LIBSSH: - if (info =3D=3D NULL) - errmsg =3D _("libssh transport error"); - else - errmsg =3D _("libssh transport error: %s"); - break; - case VIR_ERR_DEVICE_MISSING: - if (info =3D=3D NULL) - errmsg =3D _("device not found"); - else - errmsg =3D _("device not found: %s"); - break; - case VIR_ERR_INVALID_NWFILTER_BINDING: - if (info =3D=3D NULL) - errmsg =3D _("Invalid network filter binding"); - else - errmsg =3D _("Invalid network filter binding: %s"); - break; - case VIR_ERR_NO_NWFILTER_BINDING: - if (info =3D=3D NULL) - errmsg =3D _("Network filter binding not found"); - else - errmsg =3D _("Network filter binding not found: %s"); - break; - } - return errmsg; + if (info) + return virErrorMsgStrings[error].msginfo; + else + return virErrorMsgStrings[error].msg; } + /** * virReportErrorHelper: * diff --git a/src/util/virerrorpriv.h b/src/util/virerrorpriv.h index bc214393e6..1be40d8a51 100644 --- a/src/util/virerrorpriv.h +++ b/src/util/virerrorpriv.h @@ -21,6 +21,14 @@ #ifndef __VIR_ERROR_PRIV_H__ # define __VIR_ERROR_PRIV_H__ +typedef struct { + virErrorNumber error; + const char *msg; + const char *msginfo; +} virErrorMsgTuple; + +extern const virErrorMsgTuple virErrorMsgStrings[VIR_ERR_NUMBER_LAST]; + const char * virErrorMsg(virErrorNumber error, const char *info); --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 154402850602950.47526295354214; Wed, 5 Dec 2018 08:48:26 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 2C9D330C57DA; Wed, 5 Dec 2018 16:48:22 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id EE05B60920; Wed, 5 Dec 2018 16:48:21 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id AA5843F617; Wed, 5 Dec 2018 16:48:21 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm6l9028488 for ; Wed, 5 Dec 2018 11:48:06 -0500 Received: by smtp.corp.redhat.com (Postfix) id 642B95DA60; Wed, 5 Dec 2018 16:48:06 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id B43E818BB5; Wed, 5 Dec 2018 16:48:05 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:50 +0100 Message-Id: <59e589b38238dee7fa54049697e376022573956a.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 09/10] Revert "DO NOT PUSH: Make sure that error messages are not moved around" X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.45]); Wed, 05 Dec 2018 16:48:22 +0000 (UTC) Content-Type: text/plain; charset="utf-8" This reverts commit 8bce541955c7427ee78c4f6aac7a07808640fdaa. --- tests/virerrormessages.txt | 101 ------------------------------------- tests/virerrortest.c | 25 --------- 2 files changed, 126 deletions(-) delete mode 100644 tests/virerrormessages.txt diff --git a/tests/virerrormessages.txt b/tests/virerrormessages.txt deleted file mode 100644 index 7db66fa09b..0000000000 --- a/tests/virerrormessages.txt +++ /dev/null @@ -1,101 +0,0 @@ -1-internal error-internal error: %s -2-out of memory-out of memory: %s -3-this function is not supported by the connection driver-this function is= not supported by the connection driver: %s -4-unknown host-unknown host %s -5-no connection driver available-no connection driver available for %s -6-invalid connection pointer in-invalid connection pointer in %s -7-invalid domain pointer in-invalid domain pointer in %s -8-invalid argument-invalid argument: %s -9-operation failed-operation failed: %s -10-GET operation failed-GET operation failed: %s -11-POST operation failed-POST operation failed: %s -12-got unknown HTTP error code-got unknown HTTP error code %s -13-failed to serialize S-Expr-failed to serialize S-Expr: %s -14-could not use Xen hypervisor entry-could not use Xen hypervisor entry %s -15-failed Xen syscall-failed Xen syscall %s -16-unknown OS type-unknown OS type %s -17-missing kernel information-missing kernel information: %s -18-missing root device information-missing root device information in %s -19-missing source information for device-missing source information for de= vice %s -20-missing target information for device-missing target information for de= vice %s -21-missing name information-missing name information in %s -22-missing operating system information-missing operating system informati= on for %s -23-missing devices information-missing devices information for %s -24-could not connect to Xen Store-could not connect to Xen Store %s -25-too many drivers registered-too many drivers registered in %s -26-library call failed, possibly not supported-library call %s failed, pos= sibly not supported -27-XML description is invalid or not well formed-XML error: %s -28-this domain exists already-domain %s exists already -29-operation forbidden for read only access-operation forbidden: %s -30-failed to open configuration file for reading-failed to open %s for rea= ding -31-failed to read configuration file-failed to read configuration file %s -32-failed to parse configuration file-failed to parse configuration file %s -33-configuration file syntax error-configuration file syntax error: %s -34-failed to write configuration file-failed to write configuration file: = %s -35-parser error-%s -36-invalid network pointer in-invalid network pointer in %s -37-this network exists already-network %s exists already -38-system call error-%s -39-RPC error-%s -40-GNUTLS call error-%s -41-Failed to find the network-Failed to find the network: %s -42-Domain not found-Domain not found: %s -43-Network not found-Network not found: %s -44-invalid MAC address-invalid MAC address: %s -45-authentication failed-authentication failed: %s -46-invalid storage pool pointer in-invalid storage pool pointer in %s -47-invalid storage volume pointer in-invalid storage volume pointer in %s -48-Failed to find a storage driver-Failed to find a storage driver: %s -49-Storage pool not found-Storage pool not found: %s -50-Storage volume not found-Storage volume not found: %s -51-Failed to find a node driver-Failed to find a node driver: %s -52-invalid node device pointer-invalid node device pointer in %s -53-Node device not found-Node device not found: %s -54-Security model not found-Security model not found: %s -55-Requested operation is not valid-Requested operation is not valid: %s -56-Failed to find the interface-Failed to find the interface: %s -57-Interface not found-Interface not found: %s -58-invalid interface pointer in-invalid interface pointer in %s -59-multiple matching interfaces found-multiple matching interfaces found: = %s -60-Failed to start the nwfilter driver-Failed to start the nwfilter driver= : %s -61-Invalid network filter-Invalid network filter: %s -62-Network filter not found-Network filter not found: %s -63-Error while building firewall-Error while building firewall: %s -64-Failed to find a secret storage driver-Failed to find a secret storage = driver: %s -65-Invalid secret-Invalid secret: %s -66-Secret not found-Secret not found: %s -67-unsupported configuration-unsupported configuration: %s -68-Timed out during operation-Timed out during operation: %s -69-Failed to make domain persistent after migration-Failed to make domain = persistent after migration: %s -70-Hook script execution failed-Hook script execution failed: %s -71-Invalid snapshot-Invalid snapshot: %s -72-Domain snapshot not found-Domain snapshot not found: %s -73-invalid stream pointer-invalid stream pointer in %s -74-argument unsupported-argument unsupported: %s -75-Storage pool probe failed-Storage pool probe failed: %s -76-Storage pool already built-Storage pool already built: %s -77-revert requires force-revert requires force: %s -78-operation aborted-operation aborted: %s -79-authentication cancelled-authentication cancelled: %s -80-metadata not found-metadata not found: %s -81-Unsafe migration-Unsafe migration: %s -82-numerical overflow-numerical overflow: %s -83-block copy still active-block copy still active: %s -84-Operation not supported-Operation not supported: %s -85-SSH transport error-SSH transport error: %s -86-Guest agent is not responding-Guest agent is not responding: %s -87-resource busy-resource busy: %s -88-access denied-access denied: %s -89-error from service-error from service: %s -90-this storage volume exists already-storage volume %s exists already -91-the CPU is incompatible with host CPU-the CPU is incompatible with host= CPU: %s -92-XML document failed to validate against schema-XML document failed to v= alidate against schema: %s -93-migration successfully aborted-migration successfully aborted: %s -94-authentication unavailable-authentication unavailable: %s -95-Server not found-Server not found: %s -96-Client not found-Client not found: %s -97-guest agent replied with wrong id to guest-sync command-guest agent rep= lied with wrong id to guest-sync command: %s -98-libssh transport error-libssh transport error: %s -99-device not found-device not found: %s -100-Invalid network filter binding-Invalid network filter binding: %s -101-Network filter binding not found-Network filter binding not found: %s diff --git a/tests/virerrortest.c b/tests/virerrortest.c index e985ca743b..0d0377bfa8 100644 --- a/tests/virerrortest.c +++ b/tests/virerrortest.c @@ -87,29 +87,6 @@ virErrorTestMsgs(const void *opaque ATTRIBUTE_UNUSED) } -static int -virErrorTestMsgsStable(const void *opaque ATTRIBUTE_UNUSED) -{ - virBuffer buf =3D VIR_BUFFER_INITIALIZER; - char *actual =3D NULL; - size_t i; - int ret =3D 0; - - for (i =3D 1; i < VIR_ERR_NUMBER_LAST; i++) { - virBufferAsprintf(&buf, "%zu-", i); - virBufferStrcat(&buf, virErrorMsg(i, NULL), "-", virErrorMsg(i, ""= ), "\n", NULL); - } - - actual =3D virBufferContentAndReset(&buf); - - if (virTestCompareToFile(actual, abs_srcdir "/virerrormessages.txt") <= 0) - ret =3D -1; - - VIR_FREE(actual); - return ret; -} - - static int mymain(void) { @@ -117,8 +94,6 @@ mymain(void) if (virTestRun("error message strings ", virErrorTestMsgs, NULL) < 0) ret =3D -1; - if (virTestRun("error message strings stability ", virErrorTestMsgsSta= ble, NULL) < 0) - ret =3D -1; return ret =3D=3D 0 ? EXIT_SUCCESS : EXIT_FAILURE; } --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Fri Apr 19 18:34:28 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1544028498021675.4006736701796; Wed, 5 Dec 2018 08:48:18 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 99ED97F6C5; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 654CE18BB7; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 1FA943F60E; Wed, 5 Dec 2018 16:48:14 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id wB5Gm7Qj028493 for ; Wed, 5 Dec 2018 11:48:07 -0500 Received: by smtp.corp.redhat.com (Postfix) id 642275DA9B; Wed, 5 Dec 2018 16:48:07 +0000 (UTC) Received: from angien.brq.redhat.com (unknown [10.43.2.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id B56885D96F; Wed, 5 Dec 2018 16:48:06 +0000 (UTC) From: Peter Krempa To: libvir-list@redhat.com Date: Wed, 5 Dec 2018 17:47:51 +0100 Message-Id: <5f3b789397074a1a8897a929743dbc71272b47c3.1544028281.git.pkrempa@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com Cc: Peter Krempa Subject: [libvirt] [PATCH 10/10] tests: virerror: Make sure that error messages stay in correct order X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Wed, 05 Dec 2018 16:48:15 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Since we don't look up the error message according to the error code but they have to be in the correct order in virErrorMsgStrings, we need to make sure that they stay in the correct order. Signed-off-by: Peter Krempa Reviewed-by: Erik Skultety --- tests/virerrortest.c | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/tests/virerrortest.c b/tests/virerrortest.c index 0d0377bfa8..6d333f02d0 100644 --- a/tests/virerrortest.c +++ b/tests/virerrortest.c @@ -87,6 +87,24 @@ virErrorTestMsgs(const void *opaque ATTRIBUTE_UNUSED) } +static int +virErrorTestMsgOrder(const void *opaque ATTRIBUTE_UNUSED) +{ + size_t i; + int ret =3D 0; + + for (i =3D 0; i < VIR_ERR_NUMBER_LAST; i++) { + if (i !=3D virErrorMsgStrings[i].error) { + VIR_TEST_VERBOSE("\nvirErrorMsgStrings[%zu] error code is '%d'= \n", + i, virErrorMsgStrings[i].error); + ret =3D -1; + } + } + + return ret; +} + + static int mymain(void) { @@ -94,6 +112,8 @@ mymain(void) if (virTestRun("error message strings ", virErrorTestMsgs, NULL) < 0) ret =3D -1; + if (virTestRun("error messages are in correct order ", virErrorTestMsg= Order, NULL) < 0) + ret =3D -1; return ret =3D=3D 0 ? EXIT_SUCCESS : EXIT_FAILURE; } --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list