Unlock should be run *after* EINA_MAGIC_FAIL otherwise wrong info about the error will be printed, because unlock may modify d.

SVN revision: 56209
2011-01-18 08:34:14 +00:00 · 2011-01-18 08:34:14 +00:00 · c0a800288d
parent ea869cf57e
commit c0a800288d
1 changed files with 1 additions and 1 deletions
--- a/legacy/eina/src/lib/eina_share_common.c
+++ b/legacy/eina/src/lib/eina_share_common.c
@ -111,8 +111,8 @@ static const char EINA_MAGIC_SHARE_HEAD_STR[] = "Eina Share Head";
   do {                                                          \
        if (!EINA_MAGIC_CHECK((d), _node_magic))    \
          {                                                           \
-             unlock;                                                   \
             EINA_MAGIC_FAIL((d), _node_magic);        \
+             unlock;                                                   \
          }                                                           \
     } while (0)