[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 05/15] qga: add qga_open_cloexec() helper
From: |
marcandre . lureau |
Subject: |
[PULL 05/15] qga: add qga_open_cloexec() helper |
Date: |
Fri, 27 May 2022 15:54:49 +0200 |
From: Marc-André Lureau <marcandre.lureau@redhat.com>
QGA calls qemu_open_old() in various places. Calling qemu_open() instead
isn't a great alternative, as it has special "/dev/fdset" handling and
depends on QEMU internal monitor data structures.
Instead, provide a simple helper for QGA needs, with Error* support. The
following patches will make use of it.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20220525144140.591926-6-marcandre.lureau@redhat.com>
---
qga/cutils.h | 8 ++++++++
qga/cutils.c | 33 +++++++++++++++++++++++++++++++++
qga/meson.build | 1 +
3 files changed, 42 insertions(+)
create mode 100644 qga/cutils.h
create mode 100644 qga/cutils.c
diff --git a/qga/cutils.h b/qga/cutils.h
new file mode 100644
index 0000000000..f0f30a7d28
--- /dev/null
+++ b/qga/cutils.h
@@ -0,0 +1,8 @@
+#ifndef CUTILS_H_
+#define CUTILS_H_
+
+#include "qemu/osdep.h"
+
+int qga_open_cloexec(const char *name, int flags, mode_t mode);
+
+#endif /* CUTILS_H_ */
diff --git a/qga/cutils.c b/qga/cutils.c
new file mode 100644
index 0000000000..b8e142ef64
--- /dev/null
+++ b/qga/cutils.c
@@ -0,0 +1,33 @@
+/*
+ * This work is licensed under the terms of the GNU GPL, version 2 or later.
+ * See the COPYING file in the top-level directory.
+ */
+#include "cutils.h"
+
+#include "qapi/error.h"
+
+/**
+ * qga_open_cloexec:
+ * @name: the pathname to open
+ * @flags: as in open()
+ * @mode: as in open()
+ *
+ * A wrapper for open() function which sets O_CLOEXEC.
+ *
+ * On error, -1 is returned.
+ */
+int qga_open_cloexec(const char *name, int flags, mode_t mode)
+{
+ int ret;
+
+#ifdef O_CLOEXEC
+ ret = open(name, flags | O_CLOEXEC, mode);
+#else
+ ret = open(name, flags, mode);
+ if (ret >= 0) {
+ qemu_set_cloexec(ret);
+ }
+#endif
+
+ return ret;
+}
diff --git a/qga/meson.build b/qga/meson.build
index 6d9f39bb32..35fe2229e9 100644
--- a/qga/meson.build
+++ b/qga/meson.build
@@ -65,6 +65,7 @@ qga_ss.add(files(
'commands.c',
'guest-agent-command-state.c',
'main.c',
+ 'cutils.c',
))
qga_ss.add(when: 'CONFIG_POSIX', if_true: files(
'channel-posix.c',
--
2.36.1
- [PULL 07/15] qga: throw an Error in ga_channel_open(), (continued)
- [PULL 07/15] qga: throw an Error in ga_channel_open(), marcandre . lureau, 2022/05/27
- [PULL 10/15] test/qga: use G_TEST_DIR to locate os-release test file, marcandre . lureau, 2022/05/27
- [PULL 03/15] tests: make libqmp buildable for win32, marcandre . lureau, 2022/05/27
- [PULL 12/15] qga/wixl: require Mingw_bin, marcandre . lureau, 2022/05/27
- [PULL 13/15] qga/wixl: simplify some pre-processing, marcandre . lureau, 2022/05/27
- [PULL 14/15] qga/wixl: replace QEMU_GA_MSI_MINGW_BIN_PATH with glib bindir, marcandre . lureau, 2022/05/27
- [PULL 08/15] qga: replace qemu_open_old() with qga_open_cloexec(), marcandre . lureau, 2022/05/27
- [PULL 11/15] qga/wixl: prefer variables over environment, marcandre . lureau, 2022/05/27
- [PULL 15/15] test/qga: use g_auto wherever sensible, marcandre . lureau, 2022/05/27
- [PULL 04/15] qga: flatten safe_open_or_create(), marcandre . lureau, 2022/05/27
- [PULL 05/15] qga: add qga_open_cloexec() helper,
marcandre . lureau <=
- [PULL 06/15] qga: use qga_open_cloexec() for safe_open_or_create(), marcandre . lureau, 2022/05/27
- [PULL 09/15] qga: make build_fs_mount_list() return a bool, marcandre . lureau, 2022/05/27