Message ID | 20230307114507.332309-2-rodrigo@sdfg.com.ar (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Tests for idmapped tmpfs | expand |
On Tue, Mar 07, 2023 at 12:44:57PM +0100, Rodrigo Campos wrote: > Signed-off-by: Rodrigo Campos <rodrigo@sdfg.com.ar> > --- Looks good, Reviewed-by: Christian Brauner <brauner@kernel.org>
diff --git src/vfs/utils.c src/vfs/utils.c index 8b000506..ea7536c1 100644 --- src/vfs/utils.c +++ src/vfs/utils.c @@ -129,10 +129,8 @@ static int write_id_mapping(idmap_type_t map_type, pid_t pid, const char *buf, s fret = 0; out: - if (fd >= 0) - close(fd); - if (setgroups_fd >= 0) - close(setgroups_fd); + safe_close(fd); + safe_close(setgroups_fd); return fret; }
Signed-off-by: Rodrigo Campos <rodrigo@sdfg.com.ar> --- src/vfs/utils.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)