summaryrefslogtreecommitdiff
path: root/c_src/cmd_data.c
diff options
context:
space:
mode:
authorKent Overstreet <kent.overstreet@linux.dev>2025-04-13 09:36:26 -0400
committerKent Overstreet <kent.overstreet@linux.dev>2025-04-13 21:10:24 -0400
commite99da4dddbbd17d57aecc0d287a92c378a623244 (patch)
treec34ae1c73bfdccc5c6d9be1b6c9d839337993b59 /c_src/cmd_data.c
parent7b35cb1e4a4b5fa01f9be108192af0b3036229c3 (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_data.c')
-rw-r--r--c_src/cmd_data.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/c_src/cmd_data.c b/c_src/cmd_data.c
index 302c5a8c..5a1a1485 100644
--- a/c_src/cmd_data.c
+++ b/c_src/cmd_data.c
@@ -167,7 +167,7 @@ static int cmd_data_scrub(int argc, char *argv[])
if (dev->progress_fd >= 0 &&
read(dev->progress_fd, &e, sizeof(e)) != sizeof(e)) {
- close(dev->progress_fd);
+ xclose(dev->progress_fd);
dev->progress_fd = -1;
}
@@ -186,7 +186,7 @@ static int cmd_data_scrub(int argc, char *argv[])
}
if (dev->progress_fd >= 0 && e.ret) {
- close(dev->progress_fd);
+ xclose(dev->progress_fd);
dev->progress_fd = -1;
dev->ret = e.ret;
}