Fix: report setegid()/seteuid() failure in runas

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/src/common/runas.c b/src/common/runas.c
index 4a8536c6de0eb1063ed9b4f351d1f40c37355c50..9fe6dcd58efaaa5f4c551efa0ba5e03017837543 100644 (file)
--- a/src/common/runas.c
+++ b/src/common/runas.c
@@ -528,6 +528,8 @@ int handle_one_cmd(struct run_as_worker *worker)
        if (data.gid != getegid()) {
                ret = setegid(data.gid);
                if (ret < 0) {
        if (data.gid != getegid()) {
                ret = setegid(data.gid);
                if (ret < 0) {

+                       sendret._error = true;
+                       sendret._errno = errno;

                        PERROR("setegid");
                        goto write_return;
                }
                        PERROR("setegid");
                        goto write_return;
                }


@@ -535,6 +537,8 @@ int handle_one_cmd(struct run_as_worker *worker)
        if (data.uid != prev_euid) {
                ret = seteuid(data.uid);
                if (ret < 0) {
        if (data.uid != prev_euid) {
                ret = seteuid(data.uid);
                if (ret < 0) {

+                       sendret._error = true;
+                       sendret._errno = errno;

                        PERROR("seteuid");
                        goto write_return;
                }
                        PERROR("seteuid");
                        goto write_return;
                }