mirror of
https://github.com/systemd/systemd
synced 2024-09-06 16:56:43 +00:00
![Zbigniew Jędrzejewski-Szmek](/assets/img/avatar_default.png)
-1 was used everywhere, but -EBADF or -EBADFD started being used in various places. Let's make things consistent in the new style. Note that there are two candidates: EBADF 9 Bad file descriptor EBADFD 77 File descriptor in bad state Since we're initializating the fd, we're just assigning a value that means "no fd yet", so it's just a bad file descriptor, and the first errno fits better. If instead we had a valid file descriptor that became invalid because of some operation or state change, the other errno would fit better. In some places, initialization is dropped if unnecessary.
20 lines
277 B
Plaintext
20 lines
277 B
Plaintext
/* SPDX-License-Identifier: LGPL-2.1-or-later */
|
|
@@
|
|
expression fd;
|
|
@@
|
|
- close(fd);
|
|
- fd = -EBADF;
|
|
+ fd = safe_close(fd);
|
|
@@
|
|
expression fd;
|
|
@@
|
|
- close_nointr(fd);
|
|
- fd = -EBADF;
|
|
+ fd = safe_close(fd);
|
|
@@
|
|
expression fd;
|
|
@@
|
|
- safe_close(fd);
|
|
- fd = -EBADF;
|
|
+ fd = safe_close(fd);
|