Commit f53d6c7a authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Jim Harris
Browse files

rpc: only unlink unix domain socket if *we* have created it 



We used to unlink the unix domain socket
regardless of who created it. Even we
got the "socket already exists" error
on init, we would still unlink the socket
on shutdown.

Change-Id: Ib14382d6d7e127d8f51f6544cbb063594e040879
Signed-off-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/388284


Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 5bdbe36e

lib/rpc/rpc.c

+5 −5
Original line number Diff line number Diff line
@@ -175,12 +175,12 @@ spdk_rpc_register_method(const char *method, spdk_rpc_method_handler func) 
void

spdk_rpc_close(void)

{

	if (g_jsonrpc_server) {

		if (g_rpc_listen_addr_unix.sun_path[0]) {

			/* Delete the Unix socket file */

			unlink(g_rpc_listen_addr_unix.sun_path);

		}



	if (g_jsonrpc_server) {

		spdk_jsonrpc_server_shutdown(g_jsonrpc_server);

	}

}