From 329590fbb3ce0cd6ac735b17745111b21f4573e0 Mon Sep 17 00:00:00 2001 From: Olliver Schinagl Date: Wed, 14 Aug 2024 10:19:32 +0200 Subject: [PATCH] packDev_t: Clear out uninitialized memory garbage If pDev is NULL (i.e. not found in associated devices list), only the first 2 bytes of the buffer (shortAddr) were filled (with INVALID_NODE_ADDR). All the remaining buffer contained uninitialized memory garbage. Hence the memset, to zero out the entire buffer beforehand. See https://github.com/Koenkk/zigbee2mqtt/issues/13478#issuecomment-1501085509 @slugzero Signed-off-by: Olliver Schinagl --- source/ti/zstack/mt/mt_util.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/source/ti/zstack/mt/mt_util.c b/source/ti/zstack/mt/mt_util.c index 17efb7fc0..225533ade 100644 --- a/source/ti/zstack/mt/mt_util.c +++ b/source/ti/zstack/mt/mt_util.c @@ -1525,6 +1525,8 @@ static void MT_UtilBindAddEntry(uint8_t *pBuf) ***************************************************************************************************/ static void packDev_t(uint8_t *pBuf, associated_devices_t *pDev) { + memset(pBuf, 0, sizeof(associated_devices_t)); + if (NULL == pDev) { uint16_t rtrn = INVALID_NODE_ADDR;