From ad06ad57105e663984ee06780067503cbf5a22c4 Mon Sep 17 00:00:00 2001 From: William Pitcock Date: Tue, 23 Nov 2010 08:52:18 -0600 Subject: [PATCH] libratbox: Clean up uses of strcpy(). --- libratbox/src/commio.c | 4 ++-- libratbox/src/crypt.c | 4 ++-- libratbox/src/patricia.c | 2 +- libratbox/src/ratbox_lib.c | 12 ++++++------ 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/libratbox/src/commio.c b/libratbox/src/commio.c index 17066e6..56432a5 100644 --- a/libratbox/src/commio.c +++ b/libratbox/src/commio.c @@ -1197,7 +1197,7 @@ inet_ntop4(const unsigned char *src, char *dst, unsigned int size) { if(size < 16) return NULL; - return strcpy(dst, inetntoa((const char *)src)); + return rb_strlcpy(dst, inetntoa((const char *)src), size); } /* const char * @@ -1309,7 +1309,7 @@ inet_ntop6(const unsigned char *src, char *dst, unsigned int size) { return (NULL); } - return strcpy(dst, tmp); + return rb_strlcpy(dst, tmp, size); } #endif diff --git a/libratbox/src/crypt.c b/libratbox/src/crypt.c index c6d7135..b4ef99f 100644 --- a/libratbox/src/crypt.c +++ b/libratbox/src/crypt.c @@ -1450,9 +1450,9 @@ __md5_crypt(const char *pw, const char *salt) } /* Now make the output string */ - strcpy(passwd, __md5__magic); + rb_strlcpy(passwd, __md5__magic, sizeof(passwd)); strncat(passwd, sp, sl); - strcat(passwd, "$"); + rb_strlcat(passwd, "$", sizeof(passwd)); __md5_Final(final, &ctx); diff --git a/libratbox/src/patricia.c b/libratbox/src/patricia.c index 7cb2346..438763e 100644 --- a/libratbox/src/patricia.c +++ b/libratbox/src/patricia.c @@ -71,7 +71,7 @@ prefix_toa2x(rb_prefix_t *prefix, char *buf, int buf_len, int with_len) static char tmp[6]; if(prefix == NULL) { - strcpy(buf, "(NULL)"); + rb_strlcpy(buf, "(NULL)", buf_len); return (buf); } inet_ntop(prefix->family, &prefix->add.sin, buf, buf_len); diff --git a/libratbox/src/ratbox_lib.c b/libratbox/src/ratbox_lib.c index 3873d78..028a91f 100644 --- a/libratbox/src/ratbox_lib.c +++ b/libratbox/src/ratbox_lib.c @@ -69,12 +69,6 @@ rb_ctime(const time_t t, char *buf, size_t len) #else tp = gmtime(&t); #endif - if(rb_unlikely(tp == NULL)) - { - strcpy(buf, ""); - return (buf); - } - if(buf == NULL) { p = timex; @@ -86,6 +80,12 @@ rb_ctime(const time_t t, char *buf, size_t len) tlen = len; } + if(rb_unlikely(tp == NULL)) + { + rb_strlcpy(p, "", tlen); + return (p); + } + rb_snprintf(p, tlen, "%s %s %d %02u:%02u:%02u %d", s_weekdays[tp->tm_wday], s_month[tp->tm_mon], tp->tm_mday, tp->tm_hour, tp->tm_min, tp->tm_sec, tp->tm_year + 1900);