aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoy Marples <roy@marples.name>2007-10-12 00:01:33 +0000
committerRoy Marples <roy@marples.name>2007-10-12 00:01:33 +0000
commit20a149b2c7bfd7e53d24d0665281c083d7362c67 (patch)
treeb195111871b8e05c584b71aed066c3990c5db2b3
parente3bc6666d6d5bd8922ca10652a2d04577357fa71 (diff)
malloc over fatty buffers
-rw-r--r--src/librc-daemon.c16
-rw-r--r--src/librc-depend.c4
-rw-r--r--src/librc-misc.c8
-rw-r--r--src/librc.c87
-rw-r--r--src/mountinfo.c6
-rw-r--r--src/rc-depend.c12
-rw-r--r--src/rc-misc.c7
-rw-r--r--src/rc-plugin.c10
-rw-r--r--src/rc.c6
-rw-r--r--src/runscript.c8
10 files changed, 95 insertions, 69 deletions
diff --git a/src/librc-daemon.c b/src/librc-daemon.c
index 80e5b162..8fd12867 100644
--- a/src/librc-daemon.c
+++ b/src/librc-daemon.c
@@ -241,7 +241,7 @@ static bool _match_daemon (const char *path, const char *file,
const char *mexec, const char *mname,
const char *mpidfile)
{
- char buffer[RC_LINEBUFFER];
+ char *buffer;
char *ffile = rc_strcatpaths (path, file, (char *) NULL);
FILE *fp;
int lc = 0;
@@ -257,7 +257,8 @@ static bool _match_daemon (const char *path, const char *file,
if (! mpidfile)
m += 100;
- memset (buffer, 0, sizeof (buffer));
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ memset (buffer, 0, RC_LINEBUFFER);
while ((fgets (buffer, RC_LINEBUFFER, fp))) {
int lb = strlen (buffer) - 1;
if (buffer[lb] == '\n')
@@ -277,6 +278,7 @@ static bool _match_daemon (const char *path, const char *file,
if (lc > 5)
break;
}
+ free (buffer);
fclose (fp);
free (ffile);
@@ -439,7 +441,7 @@ bool rc_service_daemons_crashed (const char *service)
struct dirent *d;
char *path;
FILE *fp;
- char buffer[RC_LINEBUFFER];
+ char *buffer = NULL;
char *exec = NULL;
char *name = NULL;
char *pidfile = NULL;
@@ -458,10 +460,13 @@ bool rc_service_daemons_crashed (const char *service)
(char *) NULL);
free (svc);
- if (! (dp = opendir (dirpath)))
+ if (! (dp = opendir (dirpath))) {
+ free (dirpath);
return (false);
+ }
- memset (buffer, 0, sizeof (buffer));
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ memset (buffer, 0, RC_LINEBUFFER);
while ((d = readdir (dp))) {
if (d->d_name[0] == '.')
@@ -542,6 +547,7 @@ bool rc_service_daemons_crashed (const char *service)
name = NULL;
}
+ free (buffer);
free (exec);
free (name);
free (dirpath);
diff --git a/src/librc-depend.c b/src/librc-depend.c
index 62ceaac7..c00f4fa8 100644
--- a/src/librc-depend.c
+++ b/src/librc-depend.c
@@ -667,7 +667,7 @@ bool rc_deptree_update (void)
rc_deptype_t *deptype = NULL;
rc_deptype_t *dt;
rc_deptype_t *last_deptype = NULL;
- char buffer[RC_LINEBUFFER];
+ char *buffer = NULL;
int len;
int i;
int j;
@@ -685,6 +685,7 @@ bool rc_deptree_update (void)
deptree = xmalloc (sizeof (rc_depinfo_t));
memset (deptree, 0, sizeof (rc_depinfo_t));
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
memset (buffer, 0, RC_LINEBUFFER);
/* Phase 2 */
@@ -776,6 +777,7 @@ bool rc_deptree_update (void)
}
pclose (fp);
+ free (buffer);
/* Phase 3 - add our providors to the tree */
for (depinfo = deptree; depinfo; depinfo = depinfo->next)
diff --git a/src/librc-misc.c b/src/librc-misc.c
index d5b66b02..9390a748 100644
--- a/src/librc-misc.c
+++ b/src/librc-misc.c
@@ -90,7 +90,7 @@ char **rc_config_load (const char *file)
{
char **list = NULL;
FILE *fp;
- char buffer[RC_LINEBUFFER];
+ char *buffer;
char *p;
char *token;
char *line;
@@ -104,6 +104,7 @@ char **rc_config_load (const char *file)
if (! (fp = fopen (file, "r")))
return (NULL);
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
while (fgets (buffer, RC_LINEBUFFER, fp)) {
p = buffer;
@@ -164,6 +165,7 @@ char **rc_config_load (const char *file)
}
free (entry);
}
+ free (buffer);
fclose (fp);
return (list);
@@ -189,7 +191,7 @@ librc_hidden_def(rc_config_value)
char **rc_config_list (const char *file)
{
FILE *fp;
- char buffer[RC_LINEBUFFER];
+ char *buffer;
char *p;
char *token;
char **list = NULL;
@@ -197,6 +199,7 @@ char **rc_config_list (const char *file)
if (! (fp = fopen (file, "r")))
return (NULL);
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
while (fgets (buffer, RC_LINEBUFFER, fp)) {
p = buffer;
@@ -214,6 +217,7 @@ char **rc_config_list (const char *file)
rc_strlist_add (&list, token);
}
}
+ free (buffer);
fclose (fp);
return (list);
diff --git a/src/librc.c b/src/librc.c
index 0995d3d6..e270bbe8 100644
--- a/src/librc.c
+++ b/src/librc.c
@@ -157,21 +157,23 @@ librc_hidden_def(rc_runlevel_list)
char *rc_runlevel_get (void)
{
FILE *fp;
- char buffer[RC_LINEBUFFER];
char *runlevel = NULL;
if ((fp = fopen (SOFTLEVEL, "r"))) {
- if (fgets (buffer, PATH_MAX, fp)) {
- int i = strlen (buffer) - 1;
- if (buffer[i] == '\n')
- buffer[i] = 0;
- runlevel = xstrdup (buffer);
- }
+ runlevel = xmalloc (sizeof (char) * PATH_MAX);
+ if (fgets (runlevel, PATH_MAX, fp)) {
+ int i = strlen (runlevel) - 1;
+ if (runlevel[i] == '\n')
+ runlevel[i] = 0;
+ } else
+ *runlevel = '\0';
fclose (fp);
}
- if (! runlevel)
+ if (! runlevel || ! *runlevel) {
+ free (runlevel);
runlevel = xstrdup (RC_LEVEL_SYSINIT);
+ }
return (runlevel);
}
@@ -272,8 +274,8 @@ librc_hidden_def(rc_service_exists)
char **rc_service_options (const char *service)
{
char *svc;
- char cmd[PATH_MAX];
- char buffer[RC_LINEBUFFER];
+ char *cmd = NULL;
+ char *buffer = NULL;
char **opts = NULL;
char *token;
char *p = buffer;
@@ -282,18 +284,19 @@ char **rc_service_options (const char *service)
if (! (svc = rc_service_resolve (service)))
return (NULL);
- snprintf (cmd, sizeof (cmd), ". '%s'; echo \"${opts}\"", svc);
+ asprintf (&cmd, ". '%s'; echo \"${opts}\"", svc);
free (svc);
- if (! (fp = popen (cmd, "r")))
- return (NULL);
-
- if (fgets (buffer, RC_LINEBUFFER, fp)) {
- if (buffer[strlen (buffer) - 1] == '\n')
- buffer[strlen (buffer) - 1] = '\0';
- while ((token = strsep (&p, " ")))
- rc_strlist_addsort (&opts, token);
+ if ((fp = popen (cmd, "r"))) {
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ if (fgets (buffer, RC_LINEBUFFER, fp)) {
+ if (buffer[strlen (buffer) - 1] == '\n')
+ buffer[strlen (buffer) - 1] = '\0';
+ while ((token = strsep (&p, " ")))
+ rc_strlist_addsort (&opts, token);
+ }
+ pclose (fp);
+ free (buffer);
}
- pclose (fp);
return (opts);
}
librc_hidden_def(rc_service_options)
@@ -301,8 +304,8 @@ librc_hidden_def(rc_service_options)
char *rc_service_description (const char *service, const char *option)
{
char *svc;
- char cmd[PATH_MAX];
- char buffer[RC_LINEBUFFER];
+ char *cmd = NULL;
+ char *buffer;
char *desc = NULL;
FILE *fp;
int i;
@@ -313,25 +316,25 @@ char *rc_service_description (const char *service, const char *option)
if (! option)
option = "";
- snprintf (cmd, sizeof (cmd), ". '%s'; echo \"${description%s%s}\"",
+ asprintf (&cmd, ". '%s'; echo \"${description%s%s}\"",
svc, option ? "_" : "", option);
free (svc);
- if (! (fp = popen (cmd, "r")))
- return (NULL);
-
- while (fgets (buffer, RC_LINEBUFFER, fp)) {
- if (! desc) {
- desc = xmalloc (strlen (buffer) + 1);
- *desc = '\0';
- } else {
- desc = xrealloc (desc, strlen (desc) + strlen (buffer) + 1);
+ if ((fp = popen (cmd, "r"))) {
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ while (fgets (buffer, RC_LINEBUFFER, fp)) {
+ if (! desc) {
+ desc = xmalloc (strlen (buffer) + 1);
+ *desc = '\0';
+ } else {
+ desc = xrealloc (desc, strlen (desc) + strlen (buffer) + 1);
+ }
+ i = strlen (desc);
+ memcpy (desc + i, buffer, strlen (buffer));
+ memset (desc + i + strlen (buffer), 0, 1);
}
- i = strlen (desc);
- memcpy (desc + i, buffer, strlen (buffer));
- memset (desc + i + strlen (buffer), 0, 1);
+ free (buffer);
+ pclose (fp);
}
-
- pclose (fp);
return (desc);
}
librc_hidden_def(rc_service_description)
@@ -518,20 +521,18 @@ librc_hidden_def(rc_service_state)
char *rc_service_value_get (const char *service, const char *option)
{
FILE *fp;
- char buffer[RC_LINEBUFFER];
+ char *buffer = NULL;
char *file = rc_strcatpaths (RC_SVCDIR, "options", service, option,
(char *) NULL);
- char *value = NULL;
if ((fp = fopen (file, "r"))) {
- memset (buffer, 0, sizeof (buffer));
- if (fgets (buffer, RC_LINEBUFFER, fp))
- value = xstrdup (buffer);
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ fgets (buffer, RC_LINEBUFFER, fp);
fclose (fp);
}
free (file);
- return (value);
+ return (buffer);
}
librc_hidden_def(rc_service_value_get)
diff --git a/src/mountinfo.c b/src/mountinfo.c
index 061ae945..ef3f636d 100644
--- a/src/mountinfo.c
+++ b/src/mountinfo.c
@@ -237,7 +237,7 @@ static struct mntent *getmntfile (const char *file)
static char **find_mounts (struct args *args)
{
FILE *fp;
- char buffer[PATH_MAX * 3];
+ char *buffer;
char *p;
char *from;
char *to;
@@ -250,7 +250,8 @@ static char **find_mounts (struct args *args)
if ((fp = fopen ("/proc/mounts", "r")) == NULL)
eerrorx ("getmntinfo: %s", strerror (errno));
- while (fgets (buffer, sizeof (buffer), fp)) {
+ buffer = xmalloc (sizeof (char) * PATH_MAX * 3);
+ while (fgets (buffer, PATH_MAX * 3, fp)) {
netdev = -1;
p = buffer;
from = strsep (&p, " ");
@@ -265,6 +266,7 @@ static char **find_mounts (struct args *args)
process_mount (&list, args, from, to, fst, opts, netdev);
}
+ free (buffer);
fclose (fp);
return (list);
diff --git a/src/rc-depend.c b/src/rc-depend.c
index 2e0d3037..8a0f1894 100644
--- a/src/rc-depend.c
+++ b/src/rc-depend.c
@@ -40,7 +40,7 @@ static char *applet = NULL;
#include "_usage.h"
#define getoptstring "t:suT" getoptstring_COMMON
static struct option longopts[] = {
- { "type", 0, NULL, 't'},
+ { "type", 1, NULL, 't'},
{ "notrace", 0, NULL, 'T'},
{ "strict", 0, NULL, 's'},
{ "update", 0, NULL, 'u'},
@@ -67,7 +67,7 @@ int rc_depend (int argc, char **argv)
bool first = true;
int i;
bool update = false;
- char *runlevel = getenv ("RC_SOFTLEVEL");
+ char *runlevel = xstrdup( getenv ("RC_SOFTLEVEL"));
int opt;
char *token;
@@ -104,12 +104,12 @@ int rc_depend (int argc, char **argv)
eerrorx ("Failed to update the dependency tree");
}
- if (! runlevel)
- runlevel = rc_runlevel_get ();
-
if (! (deptree = _rc_deptree_load ()))
eerrorx ("failed to load deptree");
+ if (! runlevel)
+ runlevel = rc_runlevel_get ();
+
while (optind < argc) {
list = NULL;
rc_strlist_add (&list, argv[optind]);
@@ -128,6 +128,7 @@ int rc_depend (int argc, char **argv)
if (! services) {
rc_strlist_free (types);
rc_deptree_free (deptree);
+ free (runlevel);
if (update)
return (EXIT_SUCCESS);
eerrorx ("no services specified");
@@ -159,5 +160,6 @@ int rc_depend (int argc, char **argv)
rc_strlist_free (services);
rc_strlist_free (depends);
rc_deptree_free (deptree);
+ free (runlevel);
return (EXIT_SUCCESS);
}
diff --git a/src/rc-misc.c b/src/rc-misc.c
index 60f8009a..1012f57c 100644
--- a/src/rc-misc.c
+++ b/src/rc-misc.c
@@ -132,7 +132,7 @@ char **env_filter (void)
static bool file_regex (const char *file, const char *regex)
{
FILE *fp;
- char buffer[RC_LINEBUFFER];
+ char *buffer;
regex_t re;
bool retval = false;
int result;
@@ -140,10 +140,12 @@ static bool file_regex (const char *file, const char *regex)
if (! (fp = fopen (file, "r")))
return (false);
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
if ((result = regcomp (&re, regex, REG_EXTENDED | REG_NOSUB)) != 0) {
fclose (fp);
- regerror (result, &re, buffer, sizeof (buffer));
+ regerror (result, &re, buffer, RC_LINEBUFFER);
fprintf (stderr, "file_regex: %s", buffer);
+ free (buffer);
return (false);
}
@@ -154,6 +156,7 @@ static bool file_regex (const char *file, const char *regex)
break;
}
}
+ free (buffer);
fclose (fp);
regfree (&re);
diff --git a/src/rc-plugin.c b/src/rc-plugin.c
index 8bd50849..1cdf5690 100644
--- a/src/rc-plugin.c
+++ b/src/rc-plugin.c
@@ -150,15 +150,16 @@ void rc_plugin_run (rc_hook_t hook, const char *value)
rc_in_plugin = true;
exit (retval);
} else {
- char buffer[RC_LINEBUFFER];
+ char *buffer;
char *token;
char *p;
ssize_t nr;
close (pfd[1]);
- memset (buffer, 0, sizeof (buffer));
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ memset (buffer, 0, RC_LINEBUFFER);
- while ((nr = read (pfd[0], buffer, sizeof (buffer))) > 0) {
+ while ((nr = read (pfd[0], buffer, RC_LINEBUFFER)) > 0) {
p = buffer;
while (*p && p - buffer < nr) {
token = strsep (&p, "=");
@@ -172,7 +173,8 @@ void rc_plugin_run (rc_hook_t hook, const char *value)
}
}
}
-
+
+ free (buffer);
close (pfd[0]);
}
}
diff --git a/src/rc.c b/src/rc.c
index 0f21fe86..773bb457 100644
--- a/src/rc.c
+++ b/src/rc.c
@@ -382,7 +382,7 @@ static int do_options (int argc, char **argv)
static char *proc_getent (const char *ent)
{
FILE *fp;
- char buffer[RC_LINEBUFFER];
+ char *buffer;
char *p;
char *value = NULL;
int i;
@@ -395,7 +395,8 @@ static char *proc_getent (const char *ent)
return (NULL);
}
- memset (buffer, 0, sizeof (buffer));
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
+ memset (buffer, 0, RC_LINEBUFFER);
if (fgets (buffer, RC_LINEBUFFER, fp) &&
(p = strstr (buffer, ent)))
{
@@ -413,6 +414,7 @@ static char *proc_getent (const char *ent)
}
} else
errno = ENOENT;
+ free (buffer);
fclose (fp);
return (value);
diff --git a/src/runscript.c b/src/runscript.c
index ee49be7e..852194bb 100644
--- a/src/runscript.c
+++ b/src/runscript.c
@@ -376,7 +376,7 @@ static bool svc_exec (const char *arg1, const char *arg2)
int flags = 0;
fd_set rset;
int s;
- char buffer[RC_LINEBUFFER];
+ char *buffer;
size_t bytes;
bool prefixed = false;
int selfd;
@@ -437,6 +437,7 @@ static bool svc_exec (const char *arg1, const char *arg2)
}
selfd = MAX (master_tty, signal_pipe[0]) + 1;
+ buffer = xmalloc (sizeof (char) * RC_LINEBUFFER);
while (1) {
FD_ZERO (&rset);
FD_SET (signal_pipe[0], &rset);
@@ -456,12 +457,13 @@ static bool svc_exec (const char *arg1, const char *arg2)
break;
if (master_tty >= 0 && FD_ISSET (master_tty, &rset)) {
- bytes = read (master_tty, buffer, sizeof (buffer));
+ bytes = read (master_tty, buffer, RC_LINEBUFFER);
write_prefix (buffer, bytes, &prefixed);
}
}
}
-
+
+ free (buffer);
close (signal_pipe[0]);
close (signal_pipe[1]);
signal_pipe[0] = signal_pipe[1] = -1;