Permalink
Browse files

fix CSSNodeList memory leak

Summary:
CSSNodeListFree does not free the list->items memory
Closes facebook/yoga#214

Differential Revision: D3683643

Pulled By: emilsjolander

fbshipit-source-id: 7f9ef19f4603d5558cc2f8302756b3bcec2f6c12
  • Loading branch information...
1 parent 54f867f commit 7b02c1de3f16eeebbbc13f5ed89d59e15b6774d5 @ymmuse ymmuse committed with Facebook Github Bot 9 Aug 8, 2016
Showing with 4 additions and 1 deletion.
  1. +4 −1 React/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; }

0 comments on commit 7b02c1d

Please sign in to comment.