Double memory free after calling socket:Destroy()

[Zekkers]

The definition for wxSocketBase::Destroy is missing %ungc (or %ungc_this ?).

This comes up as a crash during the GC loop after sock:Destroy() is called.

[pkulchenko]

@Zekkers, thank you for the report. It turned out (better late than never!) that just adding %ungc_this attribute was not sufficient, so a more complex patch was needed. If you're still using wxlua, can you check if the fix in the branch https://github.com/pkulchenko/wxlua/tree/fix-double-free-131 resolves the issue for you? Thanks!

[Zekkers]

Sorry, we shelved that project now.

Glad you fixed it though :)