aboutsummaryrefslogtreecommitdiff
path: root/src/librc
diff options
context:
space:
mode:
authorAnna (navi) Figueiredo Gomes <navi@vlhl.dev>2024-07-19 16:16:07 +0200
committerAnna (navi) Figueiredo Gomes <navi@vlhl.dev>2024-07-20 03:15:17 +0200
commitf4d79466a083fe238c9f04b4bb7e7d930fff395d (patch)
tree466e96af259f09e9aef9ba5da5ca295f449424c4 /src/librc
parentc3ccaeeddc4d92bfe34cedaa6b71cb6e830fe280 (diff)
librc-misc.c, rc.c, rc-status.c: fix memory leaks
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Diffstat (limited to 'src/librc')
-rw-r--r--src/librc/librc-misc.c59
1 files changed, 29 insertions, 30 deletions
diff --git a/src/librc/librc-misc.c b/src/librc/librc-misc.c
index 538a67c2..2fbb2d8a 100644
--- a/src/librc/librc-misc.c
+++ b/src/librc/librc-misc.c
@@ -131,8 +131,9 @@ rc_proc_getent(const char *ent RC_UNUSED)
{
#ifdef __linux__
FILE *fp;
- char *proc, *p, *value = NULL;
+ char *proc = NULL, *p, *value = NULL, *save;
size_t i, len;
+ ssize_t size;
if (!exists("/proc/cmdline"))
return NULL;
@@ -140,23 +141,21 @@ rc_proc_getent(const char *ent RC_UNUSED)
if (!(fp = fopen("/proc/cmdline", "r")))
return NULL;
- proc = NULL;
i = 0;
- if (rc_getline(&proc, &i, fp) == -1 || proc == NULL)
+ if ((size = getline(&proc, &i, fp) == -1)) {
+ free(proc);
return NULL;
-
- if (proc != NULL) {
- len = strlen(ent);
-
- while ((p = strsep(&proc, " "))) {
- if (strncmp(ent, p, len) == 0 && (p[len] == '\0' || p[len] == ' ' || p[len] == '=')) {
- p += len;
-
- if (*p == '=')
- p++;
-
- value = xstrdup(p);
- }
+ }
+ proc[size - 1] = '\0';
+ save = proc;
+
+ len = strlen(ent);
+ while ((p = strsep(&save, " "))) {
+ if (strncmp(ent, p, len) == 0 && (p[len] == '\0' || p[len] == ' ' || p[len] == '=')) {
+ p += len;
+ if (*p == '=')
+ p++;
+ value = xstrdup(p);
}
}
@@ -332,13 +331,14 @@ static RC_STRINGLIST * rc_config_directory(RC_STRINGLIST *config)
{
DIR *dp;
struct dirent *d;
- RC_STRINGLIST *rc_conf_d_files = rc_stringlist_new();
+ RC_STRINGLIST *rc_conf_d_files;
RC_STRING *fname;
RC_STRINGLIST *rc_conf_d_list;
char path[PATH_MAX];
RC_STRING *line;
if ((dp = opendir(RC_CONF_D)) != NULL) {
+ rc_conf_d_files = rc_stringlist_new();
while ((d = readdir(dp)) != NULL) {
if (fnmatch("*.conf", d->d_name, FNM_PATHNAME) == 0) {
rc_stringlist_addu(rc_conf_d_files, d->d_name);
@@ -346,22 +346,21 @@ static RC_STRINGLIST * rc_config_directory(RC_STRINGLIST *config)
}
closedir(dp);
- if (rc_conf_d_files) {
- rc_stringlist_sort(&rc_conf_d_files);
- TAILQ_FOREACH(fname, rc_conf_d_files, entries) {
- if (!fname->value)
- continue;
- sprintf(path, "%s/%s", RC_CONF_D, fname->value);
- rc_conf_d_list = rc_config_list(path);
- TAILQ_FOREACH(line, rc_conf_d_list, entries)
- if (line->value)
- rc_config_set_value(config, line->value);
- rc_stringlist_free(rc_conf_d_list);
- }
+ rc_stringlist_sort(&rc_conf_d_files);
+ TAILQ_FOREACH(fname, rc_conf_d_files, entries) {
+ if (!fname->value)
+ continue;
+ sprintf(path, "%s/%s", RC_CONF_D, fname->value);
+ rc_conf_d_list = rc_config_list(path);
+ TAILQ_FOREACH(line, rc_conf_d_list, entries)
+ if (line->value)
+ rc_config_set_value(config, line->value);
+ rc_stringlist_free(rc_conf_d_list);
}
+
+ rc_stringlist_free(rc_conf_d_files);
}
- rc_stringlist_free(rc_conf_d_files);
return config;
}