[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/8] os-posix: create and export os_set_runas()
From: |
Michael Tokarev |
Subject: |
[PATCH 2/8] os-posix: create and export os_set_runas() |
Date: |
Fri, 1 Sep 2023 13:12:56 +0300 |
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Reviewed-by: Eric Blake <eblake@redhat.com>
---
include/sysemu/os-posix.h | 1 +
os-posix.c | 23 ++++++++++++++++-------
2 files changed, 17 insertions(+), 7 deletions(-)
diff --git a/include/sysemu/os-posix.h b/include/sysemu/os-posix.h
index 65b9c94e91..d32630f9e7 100644
--- a/include/sysemu/os-posix.h
+++ b/include/sysemu/os-posix.h
@@ -50,6 +50,7 @@ void os_setup_signal_handling(void);
int os_set_daemonize(bool d);
bool is_daemonized(void);
void os_daemonize(void);
+bool os_set_runas(const char *optarg);
void os_setup_post(void);
int os_mlock(void);
diff --git a/os-posix.c b/os-posix.c
index cfcb96533c..0202bb4898 100644
--- a/os-posix.c
+++ b/os-posix.c
@@ -102,8 +102,14 @@ void os_set_proc_name(const char *s)
#endif
}
-
-static bool os_parse_runas_uid_gid(const char *optarg)
+/*
+ * Prepare to change user ID. optarg can be one of 3 forms:
+ * - a username, in which case user ID will be changed to its uid,
+ * with primary and supplementary groups set up too;
+ * - a nemeric uid, in which case only the uid will be set;
+ * - a pair of numeric uid:gid.
+ */
+bool os_set_runas(const char *optarg)
{
unsigned long lv;
const char *ep;
@@ -111,6 +117,13 @@ static bool os_parse_runas_uid_gid(const char *optarg)
gid_t got_gid;
int rc;
+ user_pwd = getpwnam(optarg);
+ if (user_pwd) {
+ user_uid = -1;
+ user_gid = -1;
+ return true;
+ }
+
rc = qemu_strtoul(optarg, &ep, 0, &lv);
got_uid = lv; /* overflow here is ID in C99 */
if (rc || *ep != ':' || got_uid != lv || got_uid == (uid_t)-1) {
@@ -137,11 +150,7 @@ int os_parse_cmd_args(int index, const char *optarg)
{
switch (index) {
case QEMU_OPTION_runas:
- user_pwd = getpwnam(optarg);
- if (user_pwd) {
- user_uid = -1;
- user_gid = -1;
- } else if (!os_parse_runas_uid_gid(optarg)) {
+ if (!os_set_runas(optarg)) {
error_report("User \"%s\" doesn't exist"
" (and is not <uid>:<gid>)",
optarg);
--
2.39.2
- [PATCH 0/8] move softmmu options processing from os-posix.c to vl.c, Michael Tokarev, 2023/09/01
- [PATCH 1/8] include/sysemu/os-posix.h: move *daemonize* declarations together, Michael Tokarev, 2023/09/01
- [PATCH 2/8] os-posix: create and export os_set_runas(),
Michael Tokarev <=
- [PATCH 3/8] os-posix.c: create and export os_set_chroot(), Michael Tokarev, 2023/09/01
- [PATCH 6/8] os-posix.c: remove unneeded #includes, Michael Tokarev, 2023/09/01
- [PATCH 8/8] util/async-teardown.c: move to softmmu/, only build it when system build is requested, Michael Tokarev, 2023/09/01
- [PATCH 4/8] os-posix.c, softmmu/vl.c: move os_parse_cmd_args() into qemu_init(), Michael Tokarev, 2023/09/01
- [PATCH 5/8] os-posix.c: move code around, Michael Tokarev, 2023/09/01
- [PATCH 7/8] softmmu/vl.c: inline include/qemu/qemu-options.h into vl.c, Michael Tokarev, 2023/09/01
- Re: [PATCH 0/8] move softmmu options processing from os-posix.c to vl.c, Paolo Bonzini, 2023/09/01