Fix: only invoke PERROR() on failure to close sessiond_socket

[lttng-tools.git] / src / lib / lttng-ctl / destruction-handle.c

diff --git a/src/lib/lttng-ctl/destruction-handle.c b/src/lib/lttng-ctl/destruction-handle.c
index e22deaa930ef86fd74f25f58df6d156d39591aec..f0ff0d4e170e0e4bf2178b0075e91047b3170a4e 100644 (file)
--- a/src/lib/lttng-ctl/destruction-handle.c
+++ b/src/lib/lttng-ctl/destruction-handle.c
@@ -405,7 +405,7 @@ enum lttng_error_code lttng_destroy_session_ext(const char *session_name,
                goto error;
        }
 
-       comm_ret = lttcomm_send_unix_sock(sessiond_socket, &lsm, sizeof(lsm));
+       comm_ret = lttcomm_send_creds_unix_sock(sessiond_socket, &lsm, sizeof(lsm));
        if (comm_ret < 0) {
                ret_code = LTTNG_ERR_FATAL;
                goto error;
@@ -420,7 +420,9 @@ enum lttng_error_code lttng_destroy_session_ext(const char *session_name,
 error:
        if (sessiond_socket >= 0) {
                ret = close(sessiond_socket);
-               PERROR("Failed to close the LTTng session daemon connection socket");
+               if (ret < 0) {
+                       PERROR("Failed to close the LTTng session daemon connection socket");
+               }
        }
        if (handle) {
                lttng_destruction_handle_destroy(handle);