From 3e84a91ac1adf120f75f8382e8c71601d3655059 Mon Sep 17 00:00:00 2001 From: Rui Abreu Ferreira Date: Fri, 2 Oct 2015 00:24:20 +0100 Subject: Allow server_start to accept a NULL argument Return 1 if the endpoint argument is NULL, server_start() can get a NULL value when using server_address_new() or vim_tempname(). Removed the function attribute. --- src/nvim/msgpack_rpc/server.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'src/nvim/msgpack_rpc/server.c') diff --git a/src/nvim/msgpack_rpc/server.c b/src/nvim/msgpack_rpc/server.c index 61b8206eb6..474e25ffeb 100644 --- a/src/nvim/msgpack_rpc/server.c +++ b/src/nvim/msgpack_rpc/server.c @@ -101,8 +101,12 @@ char *server_address_new(void) /// @returns 0 on success, 1 on a regular error, and negative errno /// on failure to bind or connect. int server_start(const char *endpoint) - FUNC_ATTR_NONNULL_ALL { + if (endpoint == NULL) { + ELOG("Attempting to start server on NULL endpoint"); + return 1; + } + SocketWatcher *watcher = xmalloc(sizeof(SocketWatcher)); socket_watcher_init(&loop, watcher, endpoint, NULL); -- cgit