On 07/01/2019 00:06, Cole Robinson wrote: > On 01/04/2019 05:56 PM, Radostin Stoyanov wrote: >> Closes #7 >> >> Reported-by: @loops >> Signed-off-by: Radostin Stoyanov <[email protected]> >> --- >> src/virtBootstrap/virt_bootstrap.py | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/src/virtBootstrap/virt_bootstrap.py >> b/src/virtBootstrap/virt_bootstrap.py >> index a0b97a2..4bfa95d 100755 >> --- a/src/virtBootstrap/virt_bootstrap.py >> +++ b/src/virtBootstrap/virt_bootstrap.py >> @@ -124,6 +124,9 @@ def bootstrap(uri, dest, >> >> if not os.path.exists(dest): >> os.makedirs(dest) >> + elif os.path.abspath(dest) == "/": # Don't overwrite root >> + logger.error("Unpack to root directory is not allowed") >> + sys.exit(1) >> elif not os.path.isdir(dest): # Show error if not directory >> logger.error("Destination path '%s' is not directory.", dest) >> sys.exit(1) > > Makes sense to me, rm has similar safety checks > > Reviewed-by: Cole Robinson <[email protected]> > Thanks, pushed now.
Radostin _______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
