fs_lock:Check the nwaiter when deleting a bucket

Summary: Fixed the problem of releasing the bucket prematurely in multi-threaded flock scenarios. A thread setlk B thread setlk_wait A thread releases lock but fails to determine if nwaiter causes the bucket to be released prematurely post B thread causes crash due to heap use after free apache#13821 Signed-off-by: chenrun1 <[email protected]>
crafcat7 · Oct 5, 2024 · 338d07b · 338d07b
1 parent e16c785
commit 338d07b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/fs/vfs/fs_lock.c b/fs/vfs/fs_lock.c
@@ -237,7 +237,7 @@ static void file_lock_delete_bucket(FAR struct file_lock_bucket_s *bucket,
    * released
    */
 
-  if (list_is_empty(&bucket->list))
+  if (list_is_empty(&bucket->list) && bucket->nwaiter == 0)
     {
       /* At this point, the file has no lock information context, so we can
        * remove it from the hash table, and the return result is 0 or 1 means