cppcheck: don't check NULL pointer before freeing them

[lttng-tools.git] / src / common / sessiond-comm / sessiond-comm.c

diff --git a/src/common/sessiond-comm/sessiond-comm.c b/src/common/sessiond-comm/sessiond-comm.c
index 8ced84734772bb213a70e9a0e5bf71d9a8515f50..d1324d53013a696cea0289711d29c69bf17be4a9 100644 (file)
--- a/src/common/sessiond-comm/sessiond-comm.c
+++ b/src/common/sessiond-comm/sessiond-comm.c
@@ -78,7 +78,7 @@ const char *lttcomm_get_readable_code(enum lttcomm_return_code code)
 {
        code = -code;
 
-       if (code < LTTCOMM_CONSUMERD_COMMAND_SOCK_READY && code > LTTCOMM_NR) {
+       if (code < LTTCOMM_CONSUMERD_COMMAND_SOCK_READY || code > LTTCOMM_NR) {
                code = LTTCOMM_NR;
        }
 
@@ -312,7 +312,5 @@ alloc_error:
 __attribute__((visibility("hidden")))
 void lttcomm_destroy_sock(struct lttcomm_sock *sock)
 {
-       if (sock != NULL) {
-               free(sock);
-       }
+       free(sock);
 }