From a61398ad90acdfa4ea0ac03f3d2b9bcbfd7a6dd3 Mon Sep 17 00:00:00 2001 From: Jordan Augé Date: Fri, 30 Sep 2022 12:29:05 +0200 Subject: refactor(hicn-light): cleanup and optimizations to UDP socket face MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Ticket: HICN-771 Change-Id: Ie84dc2f61888c3fdd7b3728b2a13cacbac503d1a Signed-off-by: Jordan Augé --- ctrl/libhicnctrl/src/modules/hicn_light/connection.c | 8 ++++---- ctrl/libhicnctrl/src/modules/hicn_light/face.c | 12 +----------- ctrl/libhicnctrl/src/modules/hicn_light/face.h | 2 ++ 3 files changed, 7 insertions(+), 15 deletions(-) (limited to 'ctrl/libhicnctrl/src/modules/hicn_light') diff --git a/ctrl/libhicnctrl/src/modules/hicn_light/connection.c b/ctrl/libhicnctrl/src/modules/hicn_light/connection.c index c7d06415e..2b3644939 100644 --- a/ctrl/libhicnctrl/src/modules/hicn_light/connection.c +++ b/ctrl/libhicnctrl/src/modules/hicn_light/connection.c @@ -95,22 +95,22 @@ static int hicnlight_connection_parse(const uint8_t *buffer, size_t size, } if (!IS_VALID_ADDRESS(item->local_address)) { - ERROR("[hc_connection_parse] Invalid address received"); + ERROR("[hc_connection_parse] Invalid local address received"); return -1; } if (!IS_VALID_PORT(ntohs(item->local_port))) { - ERROR("[hc_connection_parse] Invalid port received"); + ERROR("[hc_connection_parse] Invalid local port received"); return -1; } if (!IS_VALID_ADDRESS(item->remote_address)) { - ERROR("[hc_connection_parse] Invalid address received"); + ERROR("[hc_connection_parse] Invalid remote address received"); return -1; } if (!IS_VALID_PORT(ntohs(item->remote_port))) { - ERROR("[hc_connection_parse] Invalid port received"); + ERROR("[hc_connection_parse] Invalid remote port received"); return -1; } diff --git a/ctrl/libhicnctrl/src/modules/hicn_light/face.c b/ctrl/libhicnctrl/src/modules/hicn_light/face.c index 46adf633d..4dcd96191 100644 --- a/ctrl/libhicnctrl/src/modules/hicn_light/face.c +++ b/ctrl/libhicnctrl/src/modules/hicn_light/face.c @@ -161,15 +161,5 @@ int hc_face_to_connection(const hc_face_t *face, hc_connection_t *connection, } int hc_face_to_listener(const hc_face_t *face, hc_listener_t *listener) { - switch (face->type) { - case FACE_TYPE_HICN_LISTENER: - break; - case FACE_TYPE_TCP_LISTENER: - break; - case FACE_TYPE_UDP_LISTENER: - break; - default: - return -1; - } - return -1; /* XXX Not implemented */ + return -99; /* XXX Not implemented */ } diff --git a/ctrl/libhicnctrl/src/modules/hicn_light/face.h b/ctrl/libhicnctrl/src/modules/hicn_light/face.h index 9e1cd48c2..6f202b4ce 100644 --- a/ctrl/libhicnctrl/src/modules/hicn_light/face.h +++ b/ctrl/libhicnctrl/src/modules/hicn_light/face.h @@ -4,6 +4,8 @@ #include #include +int hc_connection_to_face(const hc_connection_t *connection, hc_face_t *face); + int hc_face_from_connection(const hc_connection_t *connection, hc_face_t *face); int hc_face_to_connection(const hc_face_t *face, hc_connection_t *connection, -- cgit 1.2.3-korg