return ACL_ERROR;
}
char* src_acl_p = alloca(src_size_guess);
- /* The actual size may be smaller than our guess? I don't know. */
- ssize_t src_size = fgetxattr(src_fd, src_name, src_acl_p, src_size_guess);
+ /* The actual size may be smaller than our guess? I don't know. The
+ return value from fgetxattr() will either be nonnegative, or
+ XATTR_ERROR (which we've already ruled out), so it's safe to cast
+ it to an unsigned size_t here to avoid a compiler warning. */
+ ssize_t src_size = fgetxattr(src_fd,
+ src_name,
+ src_acl_p,
+ (size_t)src_size_guess);
if (src_size == XATTR_ERROR) {
if (errno == ENODATA) {
/* A missing ACL isn't an error. */
return ACL_ERROR;
}
- if (fsetxattr(dst_fd, dst_name, src_acl_p, src_size, 0) == XATTR_ERROR) {
+ /* See above: src_size must be nonnegative at this point,so we cast
+ it to size_t to avoid a compiler warning. */
+ if (fsetxattr(dst_fd,
+ dst_name,
+ src_acl_p,
+ (size_t)src_size,
+ 0)
+ == XATTR_ERROR) {
perror("acl_copy_xattr (fsetxattr)");
return ACL_ERROR;
}
* - @c ACL_FAILURE - If symlinks or hard links are encountered.
* - @c ACL_ERROR - Unexpected library error.
*/
-int apply_default_acl_fds(int parent_fd,
- int fd,
- bool recursive) {
+int apply_default_acl_fds(int parent_fd, int fd, bool recursive) {
int result = ACL_SUCCESS;
/* The new ACL for this path */
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)
basename_path_copy = strdup(path);
if (basename_path_copy == NULL) {
perror("apply_default_acl (strdup)");
- return ACL_ERROR;
+ result = ACL_ERROR;
+ goto cleanup;
+ }
+
+ /* 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);
}
- 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)