X-Git-Url: http://git.vpit.fr/?a=blobdiff_plain;f=Magic.xs;h=9aabb411d9a158cd52f5ed0127f3fad00ab1226d;hb=4fb41bab55f1d514f1421f8fc4aae9da00f559ec;hp=e336abe648bb8efc737b08bb2378bdaaa433ddac;hpb=d29a774d106d145036eda849ab57f42410100008;p=perl%2Fmodules%2FVariable-Magic.git

diff --git a/Magic.xs b/Magic.xs
index e336abe..9aabb41 100644
--- a/Magic.xs
+++ b/Magic.xs
@@ -449,6 +449,15 @@ STATIC void vmg_mgwiz_free(pTHX_ MGWIZ *w) {
  if (!w)
   return;
 
+ /* We reach this point in dirty state when ptable_free() is called from the
+  * atexit cleanup callback, and that the global table still holds a live
+  * wizard. This happens before all the SV bodies are freed, so all the wizard
+  * callbacks are still alive (as they are referenced by the undead wizard).
+  * Hence it is safe to decrement their refcount. Later on, the wizard free
+  * callback itself will trigger when the wizard body is reaped, but it will
+  * be skipped as it guards against dirty state - which is good since nothing
+  * has to be done anymore at that point. */
+
  SvREFCNT_dec(w->cb_data);
  SvREFCNT_dec(w->cb_get);
  SvREFCNT_dec(w->cb_set);