summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
dcaa939)
There was an error path in apply_default_acl() that was returning
directly rather than jumping to the cleanup function where memory is
freed. Thanks, clang-tidy.
basename_path_copy = strdup(path);
if (basename_path_copy == NULL) {
perror("apply_default_acl (strdup)");
basename_path_copy = strdup(path);
if (basename_path_copy == NULL) {
perror("apply_default_acl (strdup)");
+ result = ACL_ERROR;
+ goto cleanup;
}
fd = openat(parent_fd, basename(basename_path_copy), O_NOFOLLOW);
if (fd == OPEN_ERROR) {
}
fd = openat(parent_fd, basename(basename_path_copy), O_NOFOLLOW);
if (fd == OPEN_ERROR) {