Changeset 19bd84c786fecb77531073b697aad4d3c221d15d

Timestamp:

05/07/08 21:41:01 (3 months ago)

Author:

Rémi Denis-Courmont <rem@videolan.org>

git-committer:

Rémi Denis-Courmont <rem@videolan.org> 1210189261 +0300

git-parent:

[7b4e7dc93dd9bf941bcfe2e2911bbf1de86d7bc6]

git-author:

Rémi Denis-Courmont <rem@videolan.org> 1210189248 +0300

Message:

Remove useless i_refcount == 0 case and fix a tiny race condition.

When i_refcount drops to zero (under the structure lock), the object is
removed from the objects table before the structure lock is released.
Hence, no objects with a nul refcount can ever be left in the table.

There is an unlikely race condition if another object is removed from
the table right between we unlock the structure lock but before we
return from vlc_object_get() (so that the i_middle offset gets wrong).

Files:

• src/misc/objects.c (modified) (3 diffs)

src/misc/objects.c

@@ -687 +687 @@
-    vlc_object_t **pp_objects;
-    libvlc_global_data_t *p_libvlc_global = vlc_global();
+    vlc_object_t *obj = NULL;
-
-    vlc_mutex_lock( &structure_lock );
@@ -711 +712 @@
-            {
-                /* This happens when there are only two remaining objects */
-
-
+
-                {
-                    vlc_object_yield_locked( pp_objects[i_middle+1] );
-
-
+
-                }
-                break;
-            }
-        }
- if( vlc_internals( pp_objects[i_middle] )->i_refcount > 0 )
+
-        {
-            vlc_object_yield_locked( pp_objects[i_middle] );
@@ -727 +726 @@
-            return pp_objects[i_middle];
-        }
-
-        if( i_max == 0 )
-        {
-            /* this means that i_max == i_middle, and since we have already
-             * tested pp_objects[i_middle]), p_found is properly set. */
-            break;
-        }
-    }
-
-    vlc_mutex_unlock( &structure_lock );
-NULL
+obj
-}
-