diff options
author | Glenn L McGrath <bug1@ihug.co.nz> | 2004-02-22 14:25:13 +0300 |
---|---|---|
committer | Glenn L McGrath <bug1@ihug.co.nz> | 2004-02-22 14:25:13 +0300 |
commit | df7d84cf252a9443f748d8cef3821c7230ab54b4 (patch) | |
tree | 5dee4d3775a25beb64cafecb3d9bd85702cb467e /networking | |
parent | 7fc504c6f7ecd1c64387470c38af6f903c013218 (diff) |
Patch from Vodz, cleanup memory usage, send strdup error messages to
syslog.
Diffstat (limited to 'networking')
-rw-r--r-- | networking/inetd.c | 33 |
1 files changed, 22 insertions, 11 deletions
diff --git a/networking/inetd.c b/networking/inetd.c index c930a892c..283c33bed 100644 --- a/networking/inetd.c +++ b/networking/inetd.c @@ -279,6 +279,16 @@ syslog_err_and_discard_dg(int se_socktype, const char *msg, ...) _exit(1); } +static char * inetd_strdup(const char *s) +{ + char *ms = strdup(s); + + if(ms == NULL) + syslog_err_and_discard_dg(SOCK_STREAM, "strdup: %m"); + return ms; +} + + static servtab_t *getconfigent(void) { static servtab_t serv; @@ -298,13 +308,15 @@ more: if ((cp == NULL) || (*cp == '#')) { goto more; } + /* make bind 0.0.0.0 and other zero default */ + memset((char *)sep, 0, sizeof *sep); cp_ptr = strtok_r(cp, " \t", &cp_ptr_ptr); if (cp_ptr == NULL) { /* Error */ goto more; } - sep->se_service = bb_xstrdup(cp_ptr); + sep->se_service = inetd_strdup(cp_ptr); cp_ptr = strtok_r(NULL, " \t", &cp_ptr_ptr); if (cp_ptr == NULL) { @@ -339,7 +351,7 @@ more: } sep->se_family = AF_INET; } - sep->se_proto = bb_xstrdup(cp_ptr); + sep->se_proto = inetd_strdup(cp_ptr); cp_ptr = strtok_r(NULL, " \t", &cp_ptr_ptr); if (cp_ptr == NULL) { @@ -361,14 +373,16 @@ more: /* error */ goto more; } + + sep->se_user = inetd_strdup(cp_ptr); { - char *cp_ptr2 = strchr(cp_ptr, '.'); + char *cp_ptr2 = strchr(sep->se_user, '.'); + if (cp_ptr2) { *cp_ptr2++ = '\0'; - sep->se_group = bb_xstrdup(cp_ptr2); } + sep->se_group = cp_ptr2; } - sep->se_user = bb_xstrdup(cp_ptr); cp_ptr = strtok_r(NULL, " \t", &cp_ptr_ptr); if (cp_ptr == NULL) { @@ -401,19 +415,16 @@ more: sep->se_bi = NULL; } #endif - sep->se_server = bb_xstrdup(cp_ptr); + sep->se_server = inetd_strdup(cp_ptr); argc = 0; while ((cp_ptr = strtok_r(NULL, " \t", &cp_ptr_ptr)) != NULL) { if (argc < MAXARGV) { - sep->se_argv[argc++] = cp_ptr; + sep->se_argv[argc++] = inetd_strdup(cp_ptr); } } - while (argc <= MAXARGV) { - sep->se_argv[argc++] = NULL; - } + free(cp); - //free(cp); // BUG: cp is the argv[] container; we must not free it here! return (sep); } |