fs_lock/pathbuffer:Optimize code structure

Summary:
  Adjust code logic

Signed-off-by: chenrun1 <[email protected]>

Author: crafcat7

fs/vfs/fs_lock.c

@@ -610,7 +610,7 @@ int file_getlk(FAR struct file *filep, FAR struct flock *flock)
 
 out_free:
   lib_put_pathbuffer(path);
-  return OK;
+  return ret;
 }
 
 /****************************************************************************
@@ -756,7 +756,7 @@ void file_closelk(FAR struct file *filep)
   bool deleted = false;
   int ret;
 
-  if (filep->locked == false)
+  if (!filep->locked)
     {
       return;
     }
@@ -774,7 +774,7 @@ void file_closelk(FAR struct file *filep)
        * it.
        */
 
-      goto out;
+      goto out_free;
     }
 
   nxmutex_lock(&g_protect_lock);
@@ -783,7 +783,6 @@ void file_closelk(FAR struct file *filep)
     {
       /* There is no bucket here, so we don't need to free it. */
 
-      nxmutex_unlock(&g_protect_lock);
       goto out;
     }
 
@@ -807,8 +806,9 @@ void file_closelk(FAR struct file *filep)
       file_lock_delete_bucket(bucket, path);
     }
 
-  nxmutex_unlock(&g_protect_lock);
 out:
+  nxmutex_unlock(&g_protect_lock);
+out_free:
   lib_put_pathbuffer(path);
 }
 

libs/libc/misc/lib_pathbuffer.c

@@ -39,8 +39,8 @@
 
 struct pathbuffer_s
 {
-  mutex_t lock;         /* Lock for the buffer */
-  int free_bitmap;      /* Bitmap of free buffer */
+  mutex_t lock;             /* Lock for the buffer */
+  unsigned int free_bitmap; /* Bitmap of free buffer */
   char buffer[CONFIG_LIBC_MAX_PATHBUFFER][PATH_MAX];
 };