From 39be07e4e692e981a9ff1201b29ee52f7de33da5 Mon Sep 17 00:00:00 2001 From: technyon Date: Tue, 31 Jan 2023 17:25:46 +0100 Subject: [PATCH] fix of bounds memset --- src/Dhcp.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/Dhcp.cpp b/src/Dhcp.cpp index 2bfd584b..24c4bcc7 100644 --- a/src/Dhcp.cpp +++ b/src/Dhcp.cpp @@ -26,9 +26,12 @@ int DhcpClass::beginWithDHCP(uint8_t *mac, unsigned long timeout, unsigned long void DhcpClass::reset_DHCP_lease() { // zero out _dhcpSubnetMask, _dhcpGatewayIp, _dhcpLocalIp, _dhcpDhcpServerIp, _dhcpDnsServerIp - memset(_dhcpLocalIp, 0, 20); + memset(_dhcpLocalIp, 0, sizeof(_dhcpLocalIp)); + memset(_dhcpSubnetMask, 0, sizeof(_dhcpSubnetMask)); + memset(_dhcpGatewayIp, 0, sizeof(_dhcpGatewayIp)); + memset(_dhcpDhcpServerIp, 0, sizeof(_dhcpDhcpServerIp)); + memset(_dhcpDnsServerIp, 0, sizeof(_dhcpDnsServerIp)); } - //return:0 on error, 1 if request is sent and response is received int DhcpClass::request_DHCP_lease() {