Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

utils: if the path exists check if it a directory #173

Merged
merged 7 commits into from Nov 9, 2019

Conversation

giuseppe
Copy link
Member

@giuseppe giuseppe commented Nov 9, 2019

when creating a directory, if the path already exists make sure it is a directory.

Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
when creating a directory, if the path already exists make sure it is
a directory.x

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
otherwise it won't back up on a path like chroot/a

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
no need to calculate the string length multiple times.  Write directly
the buffer with snprintf, if the buffer is not big enough fail with
ENAMETOOLONG.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
if there are multiple mounts at the old root, make sure they are all
unmounted.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
regression introduced by fe15d56

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe requested a review from rhatdan November 9, 2019 23:16
@giuseppe giuseppe merged commit 91db0f4 into containers:master Nov 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant