diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-07-05 11:29:10 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-07-10 11:39:03 +0200 |
commit | e693a9323599ed6675ea598000362a9671214147 (patch) | |
tree | d2f67282cae654d4a7b1b3e7cc5c8b55922876cb /src/tty-ask-password-agent | |
parent | 44c786f04ad40d89b0a0cf4c31612b0b2a4f3765 (diff) | |
download | systemd-e693a9323599ed6675ea598000362a9671214147.tar.gz |
Use _cleanup(free_and_erasep) where appropriate
Replaces #12959.
Diffstat (limited to 'src/tty-ask-password-agent')
-rw-r--r-- | src/tty-ask-password-agent/tty-ask-password-agent.c | 22 |
1 files changed, 7 insertions, 15 deletions
diff --git a/src/tty-ask-password-agent/tty-ask-password-agent.c b/src/tty-ask-password-agent/tty-ask-password-agent.c index 271d0bb90c..843c302bf4 100644 --- a/src/tty-ask-password-agent/tty-ask-password-agent.c +++ b/src/tty-ask-password-agent/tty-ask-password-agent.c @@ -238,13 +238,13 @@ finish: } static int send_passwords(const char *socket_name, char **passwords) { - _cleanup_free_ char *packet = NULL; + _cleanup_(erase_and_freep) char *packet = NULL; _cleanup_close_ int socket_fd = -1; union sockaddr_union sa = {}; size_t packet_length = 1; char **p, *d; ssize_t n; - int r, salen; + int salen; assert(socket_name); @@ -266,22 +266,14 @@ static int send_passwords(const char *socket_name, char **passwords) { d = stpcpy(d, *p) + 1; socket_fd = socket(AF_UNIX, SOCK_DGRAM|SOCK_CLOEXEC, 0); - if (socket_fd < 0) { - r = log_debug_errno(errno, "socket(): %m"); - goto finish; - } + if (socket_fd < 0) + return log_debug_errno(errno, "socket(): %m"); n = sendto(socket_fd, packet, packet_length, MSG_NOSIGNAL, &sa.sa, salen); - if (n < 0) { - r = log_debug_errno(errno, "sendto(): %m"); - goto finish; - } + if (n < 0) + return log_debug_errno(errno, "sendto(): %m"); - r = (int) n; - -finish: - explicit_bzero_safe(packet, packet_length); - return r; + return (int) n; } static int parse_password(const char *filename, char **wall) { |