From bb617311f4fc4ece5bf963d3f445e73e09a9cdaa Mon Sep 17 00:00:00 2001 From: Dimitri Staessens Date: Sun, 25 Feb 2024 12:00:11 +0100 Subject: lib: Rename CACEP to CEP and set conngmr timeout The Common Application Connection Establishment Protocol (CACEP) is a RINA construct associated with the Common Distributed Application Protocol (CDAP). We dropped CDAP as O7s sees connection establishment as common to all applications (though it can be a nop). The wiki already refers to this as (O7s) Connection Establishment Protocol (CEP). The connection manager will now timeout waiting for CEP messages to avoid hanging forever, configurable at build time via CONNMGR_RCV_TIMEOUT. Signed-off-by: Dimitri Staessens Signed-off-by: Sander Vrijders --- src/lib/CMakeLists.txt | 8 ++-- src/lib/cacep.c | 126 ------------------------------------------------- src/lib/cep.c | 120 ++++++++++++++++++++++++++++++++++++++++++++++ src/lib/dev.c | 14 +++--- src/lib/pb/cacep.proto | 39 --------------- src/lib/pb/cep.proto | 39 +++++++++++++++ 6 files changed, 170 insertions(+), 176 deletions(-) delete mode 100644 src/lib/cacep.c create mode 100644 src/lib/cep.c delete mode 100644 src/lib/pb/cacep.proto create mode 100644 src/lib/pb/cep.proto (limited to 'src/lib') diff --git a/src/lib/CMakeLists.txt b/src/lib/CMakeLists.txt index 66f26125..a6d7ac98 100644 --- a/src/lib/CMakeLists.txt +++ b/src/lib/CMakeLists.txt @@ -10,8 +10,8 @@ protobuf_generate_c(IPCP_CONFIG_PROTO_SRCS IPCP_CONFIG_PROTO_HDRS pb/ipcp_config.proto) protobuf_generate_c(ENROLL_PROTO_SRCS ENROLL_PROTO_HDRS pb/enroll.proto) -protobuf_generate_c(CACEP_PROTO_SRCS CACEP_PROTO_HDRS - pb/cacep.proto) +protobuf_generate_c(CEP_PROTO_SRCS CEP_PROTO_HDRS + pb/cep.proto) protobuf_generate_c(IRM_PROTO_SRCS IRM_PROTO_HDRS pb/irm.proto) protobuf_generate_c(IPCP_PROTO_SRCS IPCP_PROTO_HDRS @@ -241,7 +241,7 @@ endif () set(SOURCE_FILES_DEV # Add source files here - cacep.c + cep.c dev.c ) @@ -282,7 +282,7 @@ add_library(ouroboros-common SHARED ${SOURCE_FILES_COMMON} ${IRM_PROTO_SRCS} ${IPCP_PROTO_SRCS} ${IPCP_CONFIG_PROTO_SRCS} ${MODEL_PROTO_SRCS} ${ENROLL_PROTO_SRCS}) -add_library(ouroboros-dev SHARED ${SOURCE_FILES_DEV} ${CACEP_PROTO_SRCS}) +add_library(ouroboros-dev SHARED ${SOURCE_FILES_DEV} ${CEP_PROTO_SRCS}) add_library(ouroboros-irm SHARED ${SOURCE_FILES_IRM}) diff --git a/src/lib/cacep.c b/src/lib/cacep.c deleted file mode 100644 index da1c2e2c..00000000 --- a/src/lib/cacep.c +++ /dev/null @@ -1,126 +0,0 @@ -/* - * Ouroboros - Copyright (C) 2016 - 2024 - * - * The Common Application Connection Establishment Protocol - * - * Dimitri Staessens - * Sander Vrijders - * - * This library is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public License - * version 2.1 as published by the Free Software Foundation. - * - * This library is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library; if not, write to the Free Software - * Foundation, Inc., http://www.fsf.org/about/contact/. - */ - -#define _POSIX_C_SOURCE 199309L - -#include -#include -#include - -#include -#include - -#include "cacep.pb-c.h" -typedef CacepMsg cacep_msg_t; - -#define BUF_SIZE 128 - -static int read_msg(int fd, - struct conn_info * info) -{ - uint8_t buf[BUF_SIZE]; - cacep_msg_t * msg; - ssize_t len; - - len = flow_read(fd, buf, BUF_SIZE); - if (len < 0) - return -1; - - msg = cacep_msg__unpack(NULL, len, buf); - if (msg == NULL) - return -1; - - if (strlen(msg->comp_name) > CACEP_BUF_STRLEN) { - cacep_msg__free_unpacked(msg, NULL); - return -1; - } - - strcpy(info->comp_name, msg->comp_name); - strcpy(info->protocol, msg->protocol); - - info->pref_version = msg->pref_version; - info->pref_syntax = msg->pref_syntax; - info->addr = msg->address; - - cacep_msg__free_unpacked(msg, NULL); - - return 0; -} - -static int send_msg(int fd, - const struct conn_info * info) -{ - cacep_msg_t msg = CACEP_MSG__INIT; - uint8_t * data = NULL; - size_t len = 0; - - msg.comp_name = (char *) info->comp_name; - msg.protocol = (char *) info->protocol; - msg.address = info->addr; - msg.pref_version = info->pref_version; - msg.pref_syntax = info->pref_syntax; - if (msg.pref_syntax < 0) - return -1; - - len = cacep_msg__get_packed_size(&msg); - if (len == 0) - return -1; - - data = malloc(len); - if (data == NULL) - return -ENOMEM; - - cacep_msg__pack(&msg, data); - - if (flow_write(fd, data, len) < 0) { - free(data); - return -1; - } - - free(data); - - return 0; -} - -int cacep_snd(int fd, - const struct conn_info * in) -{ - if (in == NULL) - return -EINVAL; - - if (send_msg(fd, in)) - return -1; - - return 0; -} - -int cacep_rcv(int fd, - struct conn_info * out) -{ - if (out == NULL) - return -EINVAL; - - if (read_msg(fd, out)) - return -1; - - return 0; -} diff --git a/src/lib/cep.c b/src/lib/cep.c new file mode 100644 index 00000000..ba238023 --- /dev/null +++ b/src/lib/cep.c @@ -0,0 +1,120 @@ +/* + * Ouroboros - Copyright (C) 2016 - 2024 + * + * The Ouroboros Connection Establishment Protocol + * + * Dimitri Staessens + * Sander Vrijders + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public License + * version 2.1 as published by the Free Software Foundation. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., http://www.fsf.org/about/contact/. + */ + +#define _POSIX_C_SOURCE 199309L + +#include +#include +#include + +#include +#include + +#include "cep.pb-c.h" +typedef CepMsg cep_msg_t; + +#define BUF_SIZE 128 + +static int read_msg(int fd, + struct conn_info * info) +{ + uint8_t buf[BUF_SIZE]; + cep_msg_t * msg; + ssize_t len; + + len = flow_read(fd, buf, BUF_SIZE); + if (len < 0) + return (int) len; + + msg = cep_msg__unpack(NULL, len, buf); + if (msg == NULL) + return -1; + + if (strlen(msg->comp_name) > OCEP_BUF_STRLEN) { + cep_msg__free_unpacked(msg, NULL); + return -1; + } + + strcpy(info->comp_name, msg->comp_name); + strcpy(info->protocol, msg->protocol); + + info->pref_version = msg->pref_version; + info->pref_syntax = msg->pref_syntax; + info->addr = msg->address; + + cep_msg__free_unpacked(msg, NULL); + + return 0; +} + +static int send_msg(int fd, + const struct conn_info * info) +{ + cep_msg_t msg = CEP_MSG__INIT; + uint8_t * data = NULL; + size_t len = 0; + + msg.comp_name = (char *) info->comp_name; + msg.protocol = (char *) info->protocol; + msg.address = info->addr; + msg.pref_version = info->pref_version; + msg.pref_syntax = info->pref_syntax; + if (msg.pref_syntax < 0) + return -1; + + len = cep_msg__get_packed_size(&msg); + if (len == 0) + return -1; + + data = malloc(len); + if (data == NULL) + return -ENOMEM; + + cep_msg__pack(&msg, data); + + if (flow_write(fd, data, len) < 0) { + free(data); + return -1; + } + + free(data); + + return 0; +} + +int cep_snd(int fd, + const struct conn_info * in) +{ + if (in == NULL) + return -EINVAL; + + return send_msg(fd, in); +} + +int cep_rcv(int fd, + struct conn_info * out) +{ + if (out == NULL) + return -EINVAL; + + return read_msg(fd, out); +} diff --git a/src/lib/dev.c b/src/lib/dev.c index a7f20e88..92310b9e 100644 --- a/src/lib/dev.c +++ b/src/lib/dev.c @@ -28,19 +28,19 @@ #include "config.h" -#include -#include +#include +#include #include -#include #include +#include +#include #include +#include +#include #include #include #include #include -#include -#include -#include #include #include #include @@ -48,8 +48,8 @@ #include #include #include +#include #include -#include #ifdef PROC_FLOW_STATS #include #endif diff --git a/src/lib/pb/cacep.proto b/src/lib/pb/cacep.proto deleted file mode 100644 index 166e261d..00000000 --- a/src/lib/pb/cacep.proto +++ /dev/null @@ -1,39 +0,0 @@ -/* - * Ouroboros - Copyright (C) 2016 - 2024 - * - * Message for Connection Information in CACEP - * - * Dimitri Staessens - * Sander Vrijders - * - * This library is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public License - * version 2.1 as published by the Free Software Foundation. - * - * This library is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library; if not, write to the Free Software - * Foundation, Inc., http://www.fsf.org/about/contact/. - */ - -syntax = "proto2"; - -message fixed_conc_syntax_msg { - repeated uint32 fids = 1; - repeated uint32 lens = 2; -} - -message cacep_msg { - required string comp_name = 1; - required string protocol = 2; - required int32 pref_version = 3; - repeated int32 supp_version = 4; - required int32 pref_syntax = 5; - repeated int32 supp_syntax = 6; - optional fixed_conc_syntax_msg syntax_spec = 7; - required uint64 address = 8; -} diff --git a/src/lib/pb/cep.proto b/src/lib/pb/cep.proto new file mode 100644 index 00000000..d31cf4f7 --- /dev/null +++ b/src/lib/pb/cep.proto @@ -0,0 +1,39 @@ +/* + * Ouroboros - Copyright (C) 2016 - 2024 + * + * Message for Connection Information in OCEP + * + * Dimitri Staessens + * Sander Vrijders + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public License + * version 2.1 as published by the Free Software Foundation. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., http://www.fsf.org/about/contact/. + */ + +syntax = "proto2"; + +message fixed_conc_syntax_msg { + repeated uint32 fids = 1; + repeated uint32 lens = 2; +} + +message cep_msg { + required string comp_name = 1; + required string protocol = 2; + required int32 pref_version = 3; + repeated int32 supp_version = 4; + required int32 pref_syntax = 5; + repeated int32 supp_syntax = 6; + optional fixed_conc_syntax_msg syntax_spec = 7; + required uint64 address = 8; +} -- cgit v1.2.3