diff options
| author | Lee Duncan <leeman.duncan@gmail.com> | 2020-11-10 08:23:36 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-11-10 08:23:36 -0800 |
| commit | dc6ecd3854d58ce3ecce7d30b5e412aa69e86d92 (patch) | |
| tree | b3c77a69e5e645e38f7e2d8f9ed25f8be6ccd92b | |
| parent | cf4712531a95861e0d62e7f555a522b6503a7178 (diff) | |
| parent | 84843e5ffe2c70e0bef58ffc444562e59fc7d2f0 (diff) | |
| download | open-iscsi-dc6ecd3854d58ce3ecce7d30b5e412aa69e86d92.tar.gz | |
Merge pull request #230 from wenchao-hao/master
Fix memory leak in sysfs_get_str
| -rw-r--r-- | usr/sysfs.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/usr/sysfs.c b/usr/sysfs.c index 2488160..8d37c69 100644 --- a/usr/sysfs.c +++ b/usr/sysfs.c @@ -574,8 +574,12 @@ int sysfs_get_str(char *id, char *subsys, char *param, char *value, value[0] = '\0'; sysfs_value = sysfs_get_value(id, subsys, param); - if (!sysfs_value || !strlen(sysfs_value)) + if (!sysfs_value) + return EIO; + if (!strlen(sysfs_value)) { + free(sysfs_value); return EIO; + } len = strlen(sysfs_value); if (len && (sysfs_value[len - 1] == '\n')) |