aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Hall <followingthepath@gmail.com>2015-05-05 21:32:23 +1200
committerChris Hall <chris.hall@catalyst.net.nz>2015-05-08 10:08:23 +1200
commit2393074d5f8af1983f5ce0de0b1d5567ad6ae309 (patch)
tree1141486d9c7821cfe04a248b58712f183a7d13b7
parent0b03399b8b8b862ef0ef50d6a9ece6edcae15298 (diff)
downloadrneovim-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.c2
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);
}