Changeset a210bc7 in mainline for uspace/srv/fs/ext2fs/ext2fs_ops.c

Timestamp:

2011-03-09T14:03:58Z (14 years ago)

Author:

Martin Sucha <sucha14@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

84226f0

Parents:

242b4bb

Message:

ext2fs: Protect instance list with a mutex

File:

• uspace/srv/fs/ext2fs/ext2fs_ops.c (modified) (7 diffs)

uspace/srv/fs/ext2fs/ext2fs_ops.c

@@ -102 +102 @@
  */
 static LIST_INITIALIZE(instance_list);
+static FIBRIL_MUTEX_INITIALIZE(instance_list_mutex);
 
 /**
@@ -130 +131 @@
         link_t *link;
         ext2fs_instance_t *tmp;
+        
+        fibril_mutex_lock(&instance_list_mutex);
 
         if (list_empty(&instance_list)) {
                 EXT2FS_DBG("list empty");
+                fibril_mutex_unlock(&instance_list_mutex);
                 return EINVAL;
         }
@@ -141 +145 @@
                 if (tmp->devmap_handle == devmap_handle) {
                         *inst = tmp;
+                        fibril_mutex_unlock(&instance_list_mutex);
                         return EOK;
                 }
@@ -146 +151 @@
         
         EXT2FS_DBG("not found");
+        
+        fibril_mutex_unlock(&instance_list_mutex);
         return EINVAL;
 }
@@ -462 +469 @@
         ext2fs_instance_t *inst;
         
-        
-        
         /* Accept the mount options */
         char *opts;
@@ -513 +518 @@
         inst->devmap_handle = devmap_handle;
         inst->filesystem = fs;
+        
+        fibril_mutex_lock(&instance_list_mutex);
         list_append(&inst->link, &instance_list);
+        fibril_mutex_unlock(&instance_list_mutex);
         
         async_answer_0(rid, EOK);
@@ -541 +549 @@
         
         // Remove the instance from list
+        fibril_mutex_lock(&instance_list_mutex);
         list_remove(&inst->link);
+        fibril_mutex_unlock(&instance_list_mutex);
+        
         ext2_filesystem_fini(inst->filesystem);