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

commit 2e9f00d0db905c3bc7ec0dd1e520f1fcf8db0d55
Author: liqinhui <[email protected]>
AuthorDate: Tue Jan 2 19:07:45 2024 +0800

    wifisim:Fix an assignment error.
    
    Signed-off-by: liqinhui <[email protected]>
---
 drivers/net/wifi_sim.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/drivers/net/wifi_sim.c b/drivers/net/wifi_sim.c
index 5b6429f9d8..4139e39b3d 100644
--- a/drivers/net/wifi_sim.c
+++ b/drivers/net/wifi_sim.c
@@ -762,7 +762,7 @@ static int get_bss_from_file(FAR char **rbuf)
   FAR char *p;
 
   *rbuf = malloc(size * sizeof(char));
-  if (rbuf == NULL)
+  if (*rbuf == NULL)
     {
       nerr("malloc failed!\n");
       return -ENOMEM;
@@ -777,7 +777,8 @@ redo:
       if (p == NULL)
         {
           nerr("read bss faied in realloc!\n");
-          free(rbuf);
+          free(*rbuf);
+          *rbuf = NULL;
           return -ENOMEM;
         }
 
@@ -790,7 +791,7 @@ redo:
       return ret;
     }
 
-  rbuf[ret] = '\0';
+  (*rbuf)[ret] = '\0';
 
   return ret;
 }
@@ -871,7 +872,11 @@ static int wifidriver_start_connect(FAR struct wifi_sim_s 
*wifidev)
             }
 
 error:
-          free(bss_buf);
+          if (bss_buf)
+            {
+              free(bss_buf);
+            }
+
           if (ret != OK)
             {
               wifidev->state = WLAN_STA_STATE_INIT;

Reply via email to