diff options
author | Chris Hall <followingthepath@gmail.com> | 2015-05-05 21:32:23 +1200 |
---|---|---|
committer | Chris Hall <chris.hall@catalyst.net.nz> | 2015-05-08 10:08:23 +1200 |
commit | 2393074d5f8af1983f5ce0de0b1d5567ad6ae309 (patch) | |
tree | 1141486d9c7821cfe04a248b58712f183a7d13b7 | |
parent | 0b03399b8b8b862ef0ef50d6a9ece6edcae15298 (diff) | |
download | rneovim-2393074d5f8af1983f5ce0de0b1d5567ad6ae309.tar.gz rneovim-2393074d5f8af1983f5ce0de0b1d5567ad6ae309.tar.bz2 rneovim-2393074d5f8af1983f5ce0de0b1d5567ad6ae309.zip |
coverity/109019: fixing "Sizeof not portable"
suspicious_sizeof: Passing argument 8UL /* sizeof (char const **) */ to
function xcalloc and then casting the return value to char ** is
suspicious.
In this particular case sizeof (char const **) happens to be
equal to sizeof (char const *), but this is not a portable as
-rw-r--r-- | src/nvim/msgpack_rpc/server.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/nvim/msgpack_rpc/server.c b/src/nvim/msgpack_rpc/server.c index 5e3fd100da..7f7291c3b9 100644 --- a/src/nvim/msgpack_rpc/server.c +++ b/src/nvim/msgpack_rpc/server.c @@ -268,7 +268,7 @@ char **server_address_list(size_t *size) return NULL; } - char **addrs = xcalloc((size_t) servers.ga_len, sizeof(const char **)); + char **addrs = xcalloc((size_t) servers.ga_len, sizeof(const char *)); for (int i = 0; i < servers.ga_len; i++) { addrs[i] = xstrdup(((Server **)servers.ga_data)[i]->addr); } |