From b8eca5937fc110e85d426cefba4feff00a20a04a Mon Sep 17 00:00:00 2001 From: Jordan Augé Date: Wed, 23 Oct 2019 09:58:42 +0200 Subject: [HICN-350] Minor cleanup in control API MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: I5c3705aa439a1cf6bdca915f82c0c014771cc542 Signed-off-by: Jordan Augé --- hicn-light/src/hicn/config/configuration.c | 8 ++++---- hicn-light/src/hicn/config/configurationListeners.c | 10 +++++----- hicn-light/src/hicn/config/controlAddListener.c | 2 +- hicn-light/src/hicn/config/controlListListeners.c | 12 +++++++----- hicn-light/src/hicn/config/controlRemoveListener.c | 2 +- 5 files changed, 18 insertions(+), 16 deletions(-) (limited to 'hicn-light/src/hicn/config') diff --git a/hicn-light/src/hicn/config/configuration.c b/hicn-light/src/hicn/config/configuration.c index 2b63a32a0..4771c4073 100644 --- a/hicn-light/src/hicn/config/configuration.c +++ b/hicn-light/src/hicn/config/configuration.c @@ -688,10 +688,10 @@ struct iovec *configuration_ProcessConnectionList(Configuration *config, listConnectionsCommand->connid = connection_GetConnectionId(original); const char *connectionName = symbolicNameTable_GetNameByIndex(config->symbolicNameTable, connection_GetConnectionId(original)); - snprintf(listConnectionsCommand->connectionName, 16, "%s", connectionName); + snprintf(listConnectionsCommand->connectionName, SYMBOLIC_NAME_LEN, "%s", connectionName); _strlwr(listConnectionsCommand->connectionName); - snprintf(listConnectionsCommand->interfaceName, 16, "%s", ioOperations_GetInterfaceName(connection_GetIoOperations(original))); + snprintf(listConnectionsCommand->interfaceName, SYMBOLIC_NAME_LEN, "%s", ioOperations_GetInterfaceName(connection_GetIoOperations(original))); listConnectionsCommand->state = connection_IsUp(original) ? IFACE_UP : IFACE_DOWN; @@ -801,11 +801,11 @@ struct iovec *configuration_ProcessListenersList(Configuration *config, } const char * listenerName = listenerEntry->getListenerName(listenerEntry); - snprintf(listListenersCommand->listenerName, 16, "%s", listenerName); + snprintf(listListenersCommand->listenerName, SYMBOLIC_NAME_LEN, "%s", listenerName); if (listenerEntry->getEncapType(listenerEntry) == ENCAP_TCP || listenerEntry->getEncapType(listenerEntry) == ENCAP_UDP) { const char * interfaceName = listenerEntry->getInterfaceName(listenerEntry); - snprintf(listListenersCommand->interfaceName, 16, "%s", interfaceName); + snprintf(listListenersCommand->interfaceName, SYMBOLIC_NAME_LEN, "%s", interfaceName); } } diff --git a/hicn-light/src/hicn/config/configurationListeners.c b/hicn-light/src/hicn/config/configurationListeners.c index c321007e2..8abbeb781 100644 --- a/hicn-light/src/hicn/config/configurationListeners.c +++ b/hicn-light/src/hicn/config/configurationListeners.c @@ -595,11 +595,11 @@ void configurationListeners_SetupAll(const Configuration *config, uint16_t port, const Address *address = addressListGetItem(addresses, j); // Do not start on link address - char listenerName[16]; + char listenerName[SYMBOLIC_NAME_LEN]; #ifdef __ANDROID__ - snprintf(listenerName, 16, "local_%zu", i); + snprintf(listenerName, SYMBOLIC_NAME_LEN, "local_%zu", i); #else - snprintf(listenerName, 16, "local_%ld", i); + snprintf(listenerName, SYMBOLIC_NAME_LEN, "local_%ld", i); #endif if (addressGetType(address) != ADDR_LINK) { _setupListenersOnAddress(forwarder, listenerName, address, port, @@ -617,8 +617,8 @@ void configurationListeners_SetutpLocalIPv4(const Configuration *config, in_addr_t addr = inet_addr("127.0.0.1"); uint16_t network_byte_order_port = htons(port); - char listenerNameUdp[16] = "lo_udp"; - char listenerNameTcp[16] = "lo_tcp"; + char listenerNameUdp[SYMBOLIC_NAME_LEN] = "lo_udp"; + char listenerNameTcp[SYMBOLIC_NAME_LEN] = "lo_tcp"; char *loopback_interface = "lo"; _setupUdpListenerOnInet(forwarder, listenerNameUdp,(ipv4_addr_t *)&(addr), &network_byte_order_port, loopback_interface); diff --git a/hicn-light/src/hicn/config/controlAddListener.c b/hicn-light/src/hicn/config/controlAddListener.c index cfd061131..1a94ff252 100644 --- a/hicn-light/src/hicn/config/controlAddListener.c +++ b/hicn-light/src/hicn/config/controlAddListener.c @@ -120,7 +120,7 @@ static CommandReturn _CreateListener(CommandParser *parser, CommandOps *ops, } // Fill remaining payload fields - memcpy(addListenerCommand->interfaceName, interfaceName, 16); + memcpy(addListenerCommand->interfaceName, interfaceName, SYMBOLIC_NAME_LEN); addListenerCommand->listenerMode = mode; addListenerCommand->connectionType = type; addListenerCommand->port = htons((uint16_t)atoi(port)); diff --git a/hicn-light/src/hicn/config/controlListListeners.c b/hicn-light/src/hicn/config/controlListListeners.c index 5be7b0a9b..1489470a8 100644 --- a/hicn-light/src/hicn/config/controlListListeners.c +++ b/hicn-light/src/hicn/config/controlListListeners.c @@ -95,7 +95,7 @@ static CommandReturn _controlListListeners_Execute(CommandParser *parser, char *addrString = NULL; if (receivedHeader->length > 0) { - printf("%6.6s %16s %50.70s %6s %10s\n", "iface", "name", "address", "type", "interface"); + printf("%6.6s %.*s %50.70s %6s %10s\n", "iface", SYMBOLIC_NAME_LEN, "name", "address", "type", "interface"); } else { printf(" --- No entry in the list \n"); @@ -114,15 +114,17 @@ static CommandReturn _controlListListeners_Execute(CommandParser *parser, if (strcmp(listenerType[listListenersCommand->encapType], "UDP") == 0 || strcmp(listenerType[listListenersCommand->encapType], "TCP") == 0) { - parcBufferComposer_Format(composer, "%6u %16s %50.70s %6s %10s", + parcBufferComposer_Format(composer, "%6u %.*s %50.70s %6s %10s", listListenersCommand->connid, - listListenersCommand->listenerName,addrString, + SYMBOLIC_NAME_LEN, listListenersCommand->listenerName, + addrString, listenerType[listListenersCommand->encapType], listListenersCommand->interfaceName); } else { - parcBufferComposer_Format(composer, "%6u %16s %50.70s %6s", + parcBufferComposer_Format(composer, "%6u %.*s %50.70s %6s", listListenersCommand->connid, - listListenersCommand->listenerName,addrString, + SYMBOLIC_NAME_LEN, listListenersCommand->listenerName, + addrString, listenerType[listListenersCommand->encapType]); } diff --git a/hicn-light/src/hicn/config/controlRemoveListener.c b/hicn-light/src/hicn/config/controlRemoveListener.c index 50581a8d9..545e189c0 100644 --- a/hicn-light/src/hicn/config/controlRemoveListener.c +++ b/hicn-light/src/hicn/config/controlRemoveListener.c @@ -99,7 +99,7 @@ if (!utils_ValidateSymbolicName(listenerId) && parcMemory_AllocateAndClear(sizeof(remove_listener_command)); // fill payload //removeListenerCommand->listenerId = atoi(listenerId); - strncpy(removeListenerCommand->symbolicOrListenerid, listenerId, strlen(listenerId)); + snprintf(removeListenerCommand->symbolicOrListenerid, SYMBOLIC_NAME_LEN, "%s", listenerId); // send message and receive response struct iovec *response = -- cgit 1.2.3-korg