From: Pavel Rochnyack <pavel2000@ngs.ru>
Date: Tue, 26 Sep 2017 19:41:18 +0000 (+0700)
Subject: memcached: Style fix
X-Git-Tag: collectd-5.8.0~76^2~1
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=686bd66e6b1624c07f8c469b41e902d5bfc33131;p=collectd.git

memcached: Style fix
---

diff --git a/src/memcached.c b/src/memcached.c
index 36b3c9a9..1eb02a3c 100644
--- a/src/memcached.c
+++ b/src/memcached.c
@@ -67,6 +67,7 @@ static void memcached_free(void *arg) {
   if (st->fd >= 0) {
     shutdown(st->fd, SHUT_RDWR);
     close(st->fd);
+    st->fd = -1;
   }
 
   sfree(st->name);
@@ -163,9 +164,9 @@ static int memcached_connect_inet(memcached_t *st) {
     }
 
     /* Wait until connection establishes */
-    struct pollfd pollfd;
-    pollfd.fd = fd;
-    pollfd.events = POLLOUT;
+    struct pollfd pollfd = {
+        .fd = fd, .events = POLLOUT,
+    };
     do
       status = poll(&pollfd, 1, MEMCACHED_CONNECT_TIMEOUT);
     while (status < 0 && errno == EINTR);
@@ -177,9 +178,8 @@ static int memcached_connect_inet(memcached_t *st) {
 
     /* Check if all is good */
     int socket_error;
-    socklen_t socket_error_len = sizeof(socket_error);
     status = getsockopt(fd, SOL_SOCKET, SO_ERROR, (void *)&socket_error,
-                        &socket_error_len);
+                        &(socklen_t){sizeof(socket_error)});
     if (status != 0 || socket_error != 0) {
       close(fd);
       fd = -1;
@@ -219,9 +219,9 @@ static int memcached_query_daemon(char *buffer, size_t buffer_size,
     return -1;
   }
 
-  struct pollfd pollfd;
-  pollfd.fd = st->fd;
-  pollfd.events = POLLOUT;
+  struct pollfd pollfd = {
+      .fd = st->fd, .events = POLLOUT,
+  };
 
   do
     status = poll(&pollfd, 1, MEMCACHED_IO_TIMEOUT);