From faad5cdf4d2d8de599333b18779f0c06723c3641 Mon Sep 17 00:00:00 2001 From: ymmuse Date: Mon, 8 Aug 2016 04:47:45 -0700 Subject: [PATCH] fix CSSNodeList memory leak Summary: CSSNodeListFree does not free the list->items memory Closes https://github.com/facebook/css-layout/pull/214 Differential Revision: D3683643 Pulled By: emilsjolander fbshipit-source-id: 7f9ef19f4603d5558cc2f8302756b3bcec2f6c12 --- CSSLayout/CSSNodeList.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/CSSLayout/CSSNodeList.c b/CSSLayout/CSSNodeList.c index cfea3488..c25a4c12 100644 --- a/CSSLayout/CSSNodeList.c +++ b/CSSLayout/CSSNodeList.c @@ -27,7 +27,10 @@ CSSNodeListRef CSSNodeListNew(uint32_t initialCapacity) { return list; } -void CSSNodeListFree(CSSNodeListRef list) { free(list); } +void CSSNodeListFree(CSSNodeListRef list) { + free(list->items); + free(list); +} uint32_t CSSNodeListCount(CSSNodeListRef list) { return list->count; }