fs_lock:Reduce stack size

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

Author: crafcat7

fs/vfs/fs_lock.c

@@ -548,27 +548,32 @@ int file_getlk(FAR struct file *filep, FAR struct flock *flock)
 {
   FAR struct file_lock_bucket_s *bucket;
   FAR struct file_lock_s *file_lock;
-  char path[PATH_MAX];
+  FAR char *path;
   int ret;
 
   /* We need to get the unique identifier (Path) via filep */
 
+  path = lib_get_pathbuffer();
+  if (path == NULL)
+    {
+      return -ENOMEM;
+    }
+
   ret = file_lock_get_path(filep, path);
   if (ret < 0)
     {
-      return ret;
+      goto out_free;
     }
 
   /* Convert a flock to a posix lock */
 
   ret = file_lock_normalize(filep, flock, flock);
   if (ret < 0)
     {
-      return ret;
+      goto out_free;
     }
 
   nxmutex_lock(&g_protect_lock);
-
   bucket = file_lock_find_bucket(path);
   if (bucket != NULL)
     {
@@ -602,6 +607,8 @@ int file_getlk(FAR struct file *filep, FAR struct flock *flock)
       flock->l_len = flock->l_end - flock->l_start + 1;
     }
 
+out_free:
+  lib_put_pathbuffer(path);
   return OK;
 }
 
@@ -626,24 +633,30 @@ int file_setlk(FAR struct file *filep, FAR struct flock *flock,
 {
   FAR struct file_lock_bucket_s *bucket;
   FAR struct file_lock_s *file_lock;
+  FAR char *path;
   struct flock request;
-  char path[PATH_MAX];
   int ret;
 
+  path = lib_get_pathbuffer();
+  if (path == NULL)
+    {
+      return -ENOMEM;
+    }
+
   /* We need to get the unique identifier (Path) via filep */
 
   ret = file_lock_get_path(filep, path);
   if (ret < 0)
     {
-      return ret;
+      goto out_free;
     }
 
   /* Convert a flock to a posix lock */
 
   ret = file_lock_normalize(filep, flock, &request);
   if (ret < 0)
     {
-      return ret;
+      goto out_free;
     }
 
   request.l_pid = getpid();
@@ -659,17 +672,17 @@ int file_setlk(FAR struct file *filep, FAR struct flock *flock,
 
       if (request.l_type == F_UNLCK)
         {
-          nxmutex_unlock(&g_protect_lock);
-          return OK;
+          ret = OK;
+          goto out_lock;
         }
 
       /* It looks like we didn't find a bucket, let's go create one */
 
       bucket = file_lock_create_bucket(path);
       if (bucket == NULL)
         {
-          nxmutex_unlock(&g_protect_lock);
-          return -ENOMEM;
+          ret = -ENOMEM;
+          goto out_lock;
         }
     }
   else if (request.l_type != F_UNLCK)
@@ -711,7 +724,10 @@ int file_setlk(FAR struct file *filep, FAR struct flock *flock,
 
 out:
   file_lock_delete_bucket(bucket, path);
+out_lock:
   nxmutex_unlock(&g_protect_lock);
+out_free:
+  lib_put_pathbuffer(path);
   return ret;
 }
 
@@ -731,26 +747,32 @@ void file_closelk(FAR struct file *filep)
   FAR struct file_lock_bucket_s *bucket;
   FAR struct file_lock_s *file_lock;
   FAR struct file_lock_s *temp;
-  char path[PATH_MAX];
+  FAR char *path;
   bool deleted = false;
   int ret;
 
+  path = lib_get_pathbuffer();
+  if (path == NULL)
+    {
+      return;
+    }
+
   ret = file_lock_get_path(filep, path);
   if (ret < 0)
     {
       /* It isn't an error if fs doesn't support F_GETPATH, so we just end
        * it.
        */
 
-      return;
+      goto out;
     }
 
   bucket = file_lock_find_bucket(path);
   if (bucket == NULL)
     {
       /* There is no bucket here, so we don't need to free it. */
 
-      return;
+      goto out;
     }
 
   nxmutex_lock(&g_protect_lock);
@@ -774,6 +796,8 @@ void file_closelk(FAR struct file *filep)
     }
 
   nxmutex_unlock(&g_protect_lock);
+out:
+  lib_put_pathbuffer(path);
 }
 
 /****************************************************************************