Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 30 Mar 2023 16:11:00 GMT
From:      =?utf-8?Q?Corvin=20K=C3=B6hne?= <corvink@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: e37edc91b6a0 - main - bhyve: return EEXIST when adding a fwcfg item twice
Message-ID:  <202303301611.32UGB0R2038021@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by corvink:

URL: https://cgit.FreeBSD.org/src/commit/?id=e37edc91b6a0a0116005bdd0fff7c911c37adabf

commit e37edc91b6a0a0116005bdd0fff7c911c37adabf
Author:     Corvin Köhne <corvink@FreeBSD.org>
AuthorDate: 2023-03-30 05:43:28 +0000
Commit:     Corvin Köhne <corvink@FreeBSD.org>
CommitDate: 2023-03-30 16:10:31 +0000

    bhyve: return EEXIST when adding a fwcfg item twice
    
    Reviewed by:            rew
    MFC after:              1 week
    Sponsored by:           Beckhoff Automation GmbH & Co. KG
    Differential Revision:  https://reviews.freebsd.org/D39338
---
 usr.sbin/bhyve/qemu_fwcfg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/usr.sbin/bhyve/qemu_fwcfg.c b/usr.sbin/bhyve/qemu_fwcfg.c
index 2cc334190c15..1da0a42819b8 100644
--- a/usr.sbin/bhyve/qemu_fwcfg.c
+++ b/usr.sbin/bhyve/qemu_fwcfg.c
@@ -178,7 +178,7 @@ qemu_fwcfg_add_item(const uint16_t architecture, const uint16_t index,
 	if (fwcfg_item->data != NULL) {
 		warnx("%s: qemu fwcfg item exists (architecture %s index 0x%x)",
 		    __func__, arch ? "specific" : "generic", idx);
-		return (-1);
+		return (EEXIST);
 	}
 
 	/* save data of the item */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202303301611.32UGB0R2038021>