cmm: Don't close CLI socket fd. libcli already did

cmm hands the file descriptor (fd) for the socket over which we receive
CLI commands to libcli by calling cli_loop(). The latter usually closes
the fd before it returns. There are a few unlikely out-of-memory
situations in which it doesn't, but we ignore those for now. When
cli_loop() returns, cmmCliThread() calls close() on the fd again! This
is bad because, in the meantime, the kernel might have re-assigned the
same fd to a completely unrelated socket. The second close() call from
cmmCliThread() could then erroneously close that unrelated socket.

Change-Id: I864fc6dfbb934f0426c0c8765e8a26c0c55fd81b
diff --git a/cmm/src/ffcontrol.c b/cmm/src/ffcontrol.c
index a677a94..1000259 100644
--- a/cmm/src/ffcontrol.c
+++ b/cmm/src/ffcontrol.c
@@ -2370,8 +2370,6 @@
 
 		cli_loop(ctx->handle, ctx->sock2);
 		cmm_print(DEBUG_INFO, "%s: cli_loop exiting\n", __func__);
-
-		close(ctx->sock2);
 	}
 
 	cmm_print(DEBUG_INFO, "%s: exiting\n", __func__);