tty: Avoid EPERM for no-op chown().

Change-Id: I773afca3814bfcf33a5621394e21aaf0a18f655b Signed-off-by: Michał Mirosław <emmir@google.com>
checkpoint-restore · Jun 21, 2023 · a447c2c · a447c2c
1 parent 3772781
commit a447c2c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/criu/tty.c b/criu/tty.c
@@ -22,6 +22,7 @@
 #include "rst-malloc.h"
 #include "log.h"
 #include "common/list.h"
+#include "util.h"
 #include "util-pie.h"
 #include "proc_parse.h"
 #include "file-ids.h"
@@ -867,7 +868,7 @@ static int restore_tty_params(int fd, struct tty_info *info)
 	}
 
 	if (info->tie->has_uid && info->tie->has_gid) {
-		if (fchown(fd, info->tie->uid, info->tie->gid)) {
+		if (fchown_nopriv(fd, info->tie->uid, info->tie->gid)) {
 			pr_perror("Can't setup uid %d gid %d on %#x", (int)info->tie->uid, (int)info->tie->gid,
 				  info->tfe->id);
 			return -1;