From: Steven Barth <steven@midlink.org>
Date: Mon, 23 Jun 2008 20:01:34 +0000 (+0000)
Subject: * libs/util: Fixed memory leaks in threadlocal implementations
X-Git-Tag: 0.8.0~781
X-Git-Url: https://git.archive.openwrt.org/?p=project%2Fluci.git;a=commitdiff_plain;h=1e413b92510f6da7f97e2cb1bc7a1cc45ca93bda

* libs/util: Fixed memory leaks in threadlocal implementations
---

diff --git a/libs/core/luasrc/util.lua b/libs/core/luasrc/util.lua
index 516856057..e227fc730 100644
--- a/libs/core/luasrc/util.lua
+++ b/libs/core/luasrc/util.lua
@@ -281,16 +281,9 @@ function threadlocal()
 			rawset(self, thread, {})
 		end
 		rawget(self, thread)[key] = value
-		
-		-- Avoid memory leaks by removing abandoned stores
-		for k, v in pairs(self) do
-			if type(k) == "thread" and coroutine.status(k) == "dead" then
-				rawset(self, k, nil)
-			end
-		end
 	end
 	
-	setmetatable(tbl, {__index = get, __newindex = set})
+	setmetatable(tbl, {__index = get, __newindex = set, __mode = "k"})
 	
 	return tbl
 end
@@ -404,6 +397,7 @@ end
 local performResume, handleReturnValue
 local oldpcall, oldxpcall = pcall, xpcall
 coxpt = {}
+setmetatable(coxpt, {__mode = "kv"})
 
 function handleReturnValue(err, co, status, ...)
     if not status then
@@ -429,6 +423,7 @@ function coxpcall(f, err, ...)
     end
     local c = coroutine.running()
     coxpt[co] = coxpt[c] or c or 0
+    
     return performResume(err, co, ...)
 end