Mirrors/criu
1
0
Fork 0
mirror of https://github.com/checkpoint-restore/criu.git synced 2026-01-23 02:14:37 +00:00
Code Issues Projects Releases Wiki Activity

namespace: skip cleaning up the uid/gid map in error cases

Browse source 
free_userns_maps is called to clean up uid/gid map when the dump
finishes. If we try to clean up these maps in error cases, it can lead
to double free panic. So just skip cleaning up these maps and let
free_userns_maps do its job.

Signed-off-by: Bui Quang Minh <minhquangbui99@gmail.com>
This commit is contained in:
Bui Quang Minh 2025-03-19 23:19:31 +07:00 • committed by Andrei Vagin
parent 6826ac58ce
commit 0f64709442
1 changed files with 10 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

25
criu/namespaces.c
View file

@ -1009,36 +1009,31 @@ int dump_user_ns(pid_t pid, int ns_id)
ret = parse_id_map(pid, "uid_map", &e->uid_map);
if (ret < 0)
goto err;
/*
* The uid_map and gid_map is clean up in free_userns_maps
* later, so we don't need to clean these up in error cases.
*/
return -1;
e->n_uid_map = ret;
ret = parse_id_map(pid, "gid_map", &e->gid_map);
if (ret < 0)
goto err;
return -1;
e->n_gid_map = ret;
if (check_user_ns(pid))
goto err;
return -1;
img = open_image(CR_FD_USERNS, O_DUMP, ns_id);
if (!img)
goto err;
return -1;
ret = pb_write_one(img, e, PB_USERNS);
close_image(img);
if (ret < 0)
goto err;
return -1;
return 0;
err:
if (e->uid_map) {
xfree(e->uid_map[0]);
xfree(e->uid_map);
}
if (e->gid_map) {
xfree(e->gid_map[0]);
xfree(e->gid_map);
}
return -1;
}
void free_userns_maps(void)