diff options
author | Kent Overstreet <kent.overstreet@linux.dev> | 2025-04-13 09:36:26 -0400 |
---|---|---|
committer | Kent Overstreet <kent.overstreet@linux.dev> | 2025-04-13 21:10:24 -0400 |
commit | e99da4dddbbd17d57aecc0d287a92c378a623244 (patch) | |
tree | c34ae1c73bfdccc5c6d9be1b6c9d839337993b59 /c_src/cmd_attr.c | |
parent | 7b35cb1e4a4b5fa01f9be108192af0b3036229c3 (diff) |
xclose()
Add a helper to check for close errrors - especially bad file
descriptors, that can be a fun source of heisenbugs.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'c_src/cmd_attr.c')
-rw-r--r-- | c_src/cmd_attr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/c_src/cmd_attr.c b/c_src/cmd_attr.c index 1da41265..78df1be8 100644 --- a/c_src/cmd_attr.c +++ b/c_src/cmd_attr.c @@ -48,7 +48,7 @@ static void propagate_recurse(int dirfd) continue; } propagate_recurse(fd); - close(fd); + xclose(fd); } if (errno) @@ -80,7 +80,7 @@ static void do_setattr(char *path, struct bch_opt_strs opts) die("error opening %s: %m", path); propagate_recurse(dirfd); - close(dirfd); + xclose(dirfd); } static void setattr_usage(void) |