diff options
author | Niels de Vos <ndevos@redhat.com> | 2017-01-26 16:45:51 +0100 |
---|---|---|
committer | Raghavendra G <rgowdapp@redhat.com> | 2017-01-31 00:04:44 -0500 |
commit | f2ca301bd741e3e3f076cd3f72fcd377bcef2a1a (patch) | |
tree | b729d719b29a80faaee6de85ccfb90f235c9fbb3 /rpc | |
parent | 1a95fc3036db51b82b6a80952f0908bc2019d24a (diff) |
refcount: typecast function for calling on free
All of the functions called to free the refcounted structure are doing a
typecast from (void*) to their own type taht is being free'd. This
really is not needed and the refcount interface is made a little simpler
without the requirement of typecasting.
With this small improvement in the API, all callers are updated too.
Change-Id: I32473b6d1799f62861d4b2d78ea30c09e6c80ab1
BUG: 1416889
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: https://review.gluster.org/16471
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Xavier Hernandez <xhernandez@datalab.es>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'rpc')
-rw-r--r-- | rpc/rpc-transport/socket/src/socket.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/rpc/rpc-transport/socket/src/socket.c b/rpc/rpc-transport/socket/src/socket.c index e214c772aa6..b8815773f41 100644 --- a/rpc/rpc-transport/socket/src/socket.c +++ b/rpc/rpc-transport/socket/src/socket.c @@ -3841,11 +3841,9 @@ init_openssl_mt (void) SSL_load_error_strings(); } -void -socket_poller_mayday (void *data) +static void +socket_poller_mayday (socket_private_t *priv) { - socket_private_t *priv = (socket_private_t *)data; - if (priv == NULL) return; |