diff --git a/freemodbus/tcp_master/port/port_tcp_master.c b/freemodbus/tcp_master/port/port_tcp_master.c index f0d3367..665d039 100644 --- a/freemodbus/tcp_master/port/port_tcp_master.c +++ b/freemodbus/tcp_master/port/port_tcp_master.c @@ -236,8 +236,6 @@ static BOOL xMBTCPPortMasterCloseConnection(MbSlaveInfo_t *pxInfo) static void xMBTCPPortMasterShutdown(void) { xSemaphoreGive(xShutdownSema); - vTaskDelete(NULL); - xMbPortConfig.xMbTcpTaskHandle = NULL; for (USHORT ucCnt = 0; ucCnt < MB_TCP_PORT_MAX_CONN; ucCnt++) { MbSlaveInfo_t* pxInfo = xMbPortConfig.pxMbSlaveInfo[ucCnt]; @@ -251,6 +249,8 @@ static void xMBTCPPortMasterShutdown(void) } } free(xMbPortConfig.pxMbSlaveInfo); + vTaskDelete(NULL); + xMbPortConfig.xMbTcpTaskHandle = NULL; } void vMBTCPPortMasterSetNetOpt(void *pvNetIf, eMBPortIpVer xIpVersion, eMBPortProto xProto) @@ -754,6 +754,8 @@ static void vMBTCPPortMasterTask(void *pvParameters) break; } putchar(ucDot); + // if we don't yield we run the risk of hogging CPU + vTaskDelay(pdMS_TO_TICKS(MB_TCP_CONNECTION_TIMEOUT_MS)); xErr = xMBTCPPortMasterConnect(pxInfo); switch(xErr) {