sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

drivers/sensors/apds9960.c: Fix resource leak in error handling

Browse Source 
Signed-off-by: Mingjie Shen <shen497@purdue.edu>
This commit is contained in:
Mingjie Shen 2023-06-25 23:29:15 -04:00 committed by Alan Carvalho de Assis
parent df6d799577
commit 3344697afc
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/sensors/apds9960.c
View File

@ -1218,6 +1218,7 @@ int apds9960_register(FAR const char *devpath,
if (ret != OK) if (ret != OK)
{ {
snerr("ERROR: APDS-9960 is not responding!\n"); snerr("ERROR: APDS-9960 is not responding!\n");
kmm_free(priv);
return ret; return ret;
} }
@ -1227,6 +1228,7 @@ int apds9960_register(FAR const char *devpath,
if (ret < 0) if (ret < 0)
{ {
snerr("ERROR: Failed to initialize the APDS9960!\n"); snerr("ERROR: Failed to initialize the APDS9960!\n");
kmm_free(priv);
return ret; return ret;
} }
@ -1240,6 +1242,7 @@ int apds9960_register(FAR const char *devpath,
if (ret < 0) if (ret < 0)
{ {
snerr("ERROR: Failed to initialize the APDS9960!\n"); snerr("ERROR: Failed to initialize the APDS9960!\n");
kmm_free(priv);
return ret; return ret;
} }
@ -1249,6 +1252,7 @@ int apds9960_register(FAR const char *devpath,
if (ret < 0) if (ret < 0)
{ {
snerr("ERROR: Failed to initialize the APDS9960!\n"); snerr("ERROR: Failed to initialize the APDS9960!\n");
kmm_free(priv);
return ret; return ret;
} }
@ -1262,6 +1266,7 @@ int apds9960_register(FAR const char *devpath,
if (ret < 0) if (ret < 0)
{ {
snerr("ERROR: Failed to write APDS9960_GCONFIG4!\n"); snerr("ERROR: Failed to write APDS9960_GCONFIG4!\n");
kmm_free(priv);
return ret; return ret;
} }
@ -1271,6 +1276,7 @@ int apds9960_register(FAR const char *devpath,
if (ret < 0) if (ret < 0)
{ {
snerr("ERROR: Failed to initialize the APDS9960!\n"); snerr("ERROR: Failed to initialize the APDS9960!\n");
kmm_free(priv);
return ret; return ret;
} }