transport: mctp-demux: Don't test socket for non-zero value
It's valid for a file descriptor to take the value 0. It may be unusual
for the socket descriptor value to be zero, but it's technically not
invalid.
Fixes: c1b66f420912 ("requester: Add new APIs to support multiple transports")
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: Idda55c2766121842c0afc8632469e51b8a562772
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 52ae6ed..cf93e66 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -39,6 +39,7 @@
1. pdr: Return success for pldm_pdr_find_child_container_id_range_exclude() API
2. pdr: Rework pldm_pdr_find_container_id_range_exclude() API
+3. transport: mctp-demux: Don't test socket for non-zero value
## [0.3.0] - 2023-06-23
diff --git a/src/transport/mctp-demux.c b/src/transport/mctp-demux.c
index 8988889..427c52a 100644
--- a/src/transport/mctp-demux.c
+++ b/src/transport/mctp-demux.c
@@ -278,8 +278,9 @@
int pldm_transport_mctp_demux_get_socket_fd(
struct pldm_transport_mctp_demux *ctx)
{
- if (ctx && ctx->socket) {
+ if (ctx) {
return ctx->socket;
}
+
return -1;
}