X-Git-Url: http://gitweb.michael.orlitzky.com/?a=blobdiff_plain;f=src%2Flibadacl.c;h=8ff0c17de6a8a91812469880af2003fbb6e6644f;hb=cb2ef6b2386de44200124904c7a03ce8512f3585;hp=f20d4a393c26f4e22a8e1cb77947214539b6fb5e;hpb=a5d23c3d91c9a7b16e5aec08555133f9ded7d1bb;p=apply-default-acl.git diff --git a/src/libadacl.c b/src/libadacl.c index f20d4a3..8ff0c17 100644 --- a/src/libadacl.c +++ b/src/libadacl.c @@ -1003,7 +1003,19 @@ int apply_default_acl(const char* path, bool recursive) { return ACL_ERROR; } char* parent = dirname(dirname_path_copy); + + /* Just kidding, if the path is "." or "..", then dirname will do + * the wrong thing and give us "." as its parent, too. So, we handle + * those as special cases. + */ + bool path_is_dots = strcmp(path, ".") == 0 || strcmp(path, "..") == 0; + char dots_parent[6] = "../"; + if (path_is_dots) { + parent = strcat(dots_parent, path); + } + parent_fd = safe_open(parent, O_DIRECTORY | O_NOFOLLOW); + if (parent_fd == OPEN_ERROR) { if (errno == ELOOP || errno == ENOTDIR) { /* We hit a symlink, either in the last path component (ELOOP) @@ -1028,7 +1040,16 @@ int apply_default_acl(const char* path, bool recursive) { result = ACL_ERROR; goto cleanup; } - fd = openat(parent_fd, basename(basename_path_copy), O_NOFOLLOW); + + /* If the basename is "." or "..", then we don't want to open it + relative to the parent_fd, so we need another special case for + those paths. */ + if (path_is_dots) { + fd = open(path, O_NOFOLLOW); + } + else { + fd = openat(parent_fd, basename(basename_path_copy), O_NOFOLLOW); + } if (fd == OPEN_ERROR) { if (errno == ELOOP || errno == ENOTDIR) { /* We hit a symlink, either in the last path component (ELOOP)