(nuttx) branch master updated: fs_automount:fix ci break

[xiaoxiang]

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new 8068ba75ec fs_automount:fix ci break
8068ba75ec is described below

commit 8068ba75ec4e24d0f553a2d38a2f4785158fc81e
Author: chenrun1 <chenr...@xiaomi.com>
AuthorDate: Wed Nov 13 10:01:06 2024 +0800

    fs_automount:fix ci break
    
    
====================================================================================
    Configuration/Tool: spresense/example_camera,CONFIG_ARM_TOOLCHAIN_GNU_EABI
    2024-11-12 12:24:37
    
------------------------------------------------------------------------------------
      Cleaning...
      Configuring...
      Disabling CONFIG_ARM_TOOLCHAIN_GNU_EABI
      Enabling CONFIG_ARM_TOOLCHAIN_GNU_EABI
      Building NuttX...
    mount/fs_automount.c: In function 'automount_initialize':
    Error: mount/fs_automount.c:816:7: error: 'return' with no value, in 
function returning non-void [-Werror=return-type]
      816 |       return;
          |       ^~~~~~
    In file included from mount/fs_automount.c:43:
    /github/workspace/sources/nuttx/include/nuttx/fs/automount.h:176:11: note: 
declared here
      176 | FAR void *automount_initialize(FAR const struct automount_lower_s 
*lower);
          |           ^~~~~~~~~~~~~~~~~~~~
    cc1: all warnings being treated as errors
    
    Signed-off-by: chenrun1 <chenr...@xiaomi.com>
---
 fs/mount/fs_automount.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/fs/mount/fs_automount.c b/fs/mount/fs_automount.c
index 47fd173206..e96f742577 100644
--- a/fs/mount/fs_automount.c
+++ b/fs/mount/fs_automount.c
@@ -813,7 +813,7 @@ FAR void *automount_initialize(FAR const struct 
automount_lower_s *lower)
   FAR char *devpath = lib_get_pathbuffer();
   if (devpath == NULL)
     {
-      return;
+      return NULL;
     }
 #endif /* CONFIG_FS_AUTOMOUNTER_DRIVER */
 
@@ -826,6 +826,9 @@ FAR void *automount_initialize(FAR const struct 
automount_lower_s *lower)
   if (priv == NULL)
     {
       ferr("ERROR: Failed to allocate state structure\n");
+#ifdef CONFIG_FS_AUTOMOUNTER_DRIVER
+      lib_put_pathbuffer(devpath);
+#endif /* CONFIG_FS_AUTOMOUNTER_DRIVER */
       return NULL;
     }