projects
/
systemd
/
.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
8d51633
)
sd-bus: cleanup bus_close_fds()
author
Namhyung Kim
<namhyung@gmail.com>
Sat, 1 Aug 2015 15:35:22 +0000
(
00:35
+0900)
committer
Namhyung Kim
<namhyung@gmail.com>
Sun, 2 Aug 2015 14:12:08 +0000
(23:12 +0900)
The safe_close() already checks the fd and returns -1.
src/libsystemd/sd-bus/sd-bus.c
patch
|
blob
|
history
diff --git
a/src/libsystemd/sd-bus/sd-bus.c
b/src/libsystemd/sd-bus/sd-bus.c
index 767df40e81abaf832328197ab369d5f63884ba1d..b9e93f6f41e4c1651f2096dec9234f11d8e66c6c 100644
(file)
--- a/
src/libsystemd/sd-bus/sd-bus.c
+++ b/
src/libsystemd/sd-bus/sd-bus.c
@@
-73,13
+73,9
@@
static void bus_close_fds(sd_bus *b) {
detach_io_events(b);
- if (b->input_fd >= 0)
- safe_close(b->input_fd);
-
- if (b->output_fd >= 0 && b->output_fd != b->input_fd)
+ if (b->input_fd != b->output_fd)
safe_close(b->output_fd);
-
- b->input_fd = b->output_fd = -1;
+ b->output_fd = b->input_fd = safe_close(b->input_fd);
}
static void bus_reset_queues(sd_bus *b) {