diff options
author | Rich Megginson <rmeggins@redhat.com> | 2005-03-05 04:29:24 +0000 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2005-03-05 04:29:24 +0000 |
commit | b352660e243c7b9b7d050f1c38cff1c9faf278b1 (patch) | |
tree | ede08019beb931c3206609ab2377a015d510bdb4 /ldap/servers/slapd/snmp_collator.c | |
parent | f08951680ddfebc3f3df07e720ad0650fe473c0f (diff) | |
download | ds-b352660e243c7b9b7d050f1c38cff1c9faf278b1.tar.gz ds-b352660e243c7b9b7d050f1c38cff1c9faf278b1.tar.xz ds-b352660e243c7b9b7d050f1c38cff1c9faf278b1.zip |
clean up sprintf usage and many other flawfinder issues; clean up compiler warnings on Linux; remove pam_passthru from DS 7.1
Diffstat (limited to 'ldap/servers/slapd/snmp_collator.c')
-rw-r--r-- | ldap/servers/slapd/snmp_collator.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/ldap/servers/slapd/snmp_collator.c b/ldap/servers/slapd/snmp_collator.c index 68fe4423..3c0f5e65 100644 --- a/ldap/servers/slapd/snmp_collator.c +++ b/ldap/servers/slapd/snmp_collator.c @@ -234,9 +234,7 @@ char *make_ds_url(char *host, int port){ char *url; - url = (char *)slapi_ch_malloc( (strlen(host) + PORT_LEN + URL_CHARS_LEN + 1) * sizeof(char)); - - sprintf(url,"ldap://%s:%d/",host, port); + url = slapi_ch_smprintf("ldap://%s:%d/",host, port); return url; } @@ -362,7 +360,7 @@ int snmp_collator_start() * Get directory for our stats file */ - sprintf(szStatsFile, "%s/logs/%s", instancedir, + PR_snprintf(szStatsFile, sizeof(szStatsFile), "%s/logs/%s", instancedir, AGT_STATS_FILE); tmpstatsfile = szStatsFile; |