fix(previews): escape container names in stopContainers method to prevent shell injection vulnerabilities
This commit is contained in:
@@ -247,7 +247,7 @@ class Previews extends Component
|
|||||||
$containerNames[] = str_replace('/', '', $container['Names']);
|
$containerNames[] = str_replace('/', '', $container['Names']);
|
||||||
}
|
}
|
||||||
|
|
||||||
$containerList = implode(' ', $containerNames);
|
$containerList = implode(' ', array_map('escapeshellarg', $containerNames));
|
||||||
$commands = [
|
$commands = [
|
||||||
"docker stop --time=$timeout $containerList",
|
"docker stop --time=$timeout $containerList",
|
||||||
"docker rm -f $containerList",
|
"docker rm -f $containerList",
|
||||||
|
|||||||
Reference in New Issue
Block a user