]> eyrie.org Git - kerberos/wallet.git/commitdiff
Remove remaining uses of strlcpy and strlcat
authorRuss Allbery <eagle@eyrie.org>
Sun, 17 Jan 2016 22:33:11 +0000 (14:33 -0800)
committerRuss Allbery <eagle@eyrie.org>
Sun, 17 Jan 2016 22:33:11 +0000 (14:33 -0800)
Makefile.am
configure.ac
portable/strlcat.c [deleted file]
portable/strlcpy.c [deleted file]
tests/TESTS
tests/portable/strlcat-t.c [deleted file]
tests/portable/strlcat.c [deleted file]
tests/portable/strlcpy-t.c [deleted file]
tests/portable/strlcpy.c [deleted file]

index 5c876391d9e4f2be8a2887e4c763ce2ff0e2ca17..214bb84cc3939ab3167a9d3dfa38812b74961c6e 100644 (file)
@@ -290,12 +290,6 @@ tests_portable_setenv_t_LDADD = tests/tap/libtap.a portable/libportable.a
 tests_portable_snprintf_t_SOURCES = tests/portable/snprintf-t.c \
        tests/portable/snprintf.c
 tests_portable_snprintf_t_LDADD = tests/tap/libtap.a portable/libportable.a
-tests_portable_strlcat_t_SOURCES = tests/portable/strlcat-t.c \
-       tests/portable/strlcat.c
-tests_portable_strlcat_t_LDADD = tests/tap/libtap.a portable/libportable.a
-tests_portable_strlcpy_t_SOURCES = tests/portable/strlcpy-t.c \
-       tests/portable/strlcpy.c
-tests_portable_strlcpy_t_LDADD = tests/tap/libtap.a portable/libportable.a
 tests_util_messages_krb5_t_CPPFLAGS = $(KRB5_CPPFLAGS)
 tests_util_messages_krb5_t_LDFLAGS = $(KRB5_LDFLAGS)
 tests_util_messages_krb5_t_LDADD = tests/tap/libtap.a util/libutil.a \
index f4541e275b9858ca16b192acc61cd0dc9c4c322d..3b3c787813bccd6b49eb590718a4eaf4c1fbe56b 100644 (file)
@@ -61,7 +61,7 @@ RRA_LIB_KRB5_RESTORE
 dnl Probe for properties of the C library.
 AC_HEADER_STDBOOL
 AC_CHECK_HEADERS([sys/bitypes.h sys/uio.h sys/time.h syslog.h])
-AC_CHECK_DECLS([snprintf, strlcat, strlcpy, vsnprintf])
+AC_CHECK_DECLS([snprintf, vsnprintf])
 RRA_C_C99_VAMACROS
 RRA_C_GNU_VAMACROS
 AC_TYPE_LONG_LONG_INT
@@ -69,7 +69,7 @@ AC_CHECK_TYPES([ssize_t], [], [],
     [#include <sys/types.h>])
 RRA_FUNC_SNPRINTF
 AC_CHECK_FUNCS([setrlimit])
-AC_REPLACE_FUNCS([asprintf mkstemp reallocarray setenv strlcat strlcpy])
+AC_REPLACE_FUNCS([asprintf mkstemp reallocarray setenv])
 
 dnl Find a remctld binary for the test suite.
 AC_ARG_VAR([REMCTLD], [Path to the remctld binary])
diff --git a/portable/strlcat.c b/portable/strlcat.c
deleted file mode 100644 (file)
index 613d3f2..0000000
+++ /dev/null
@@ -1,52 +0,0 @@
-/*
- * Replacement for a missing strlcat.
- *
- * Provides the same functionality as the *BSD function strlcat, originally
- * developed by Todd Miller and Theo de Raadt.  strlcat works similarly to
- * strncat, except simpler.  The result is always nul-terminated even if the
- * source string is longer than the space remaining in the destination string,
- * and the total space required is returned.  The third argument is the total
- * space available in the destination buffer, not just the amount of space
- * remaining.
- *
- * The canonical version of this file is maintained in the rra-c-util package,
- * which can be found at <http://www.eyrie.org/~eagle/software/rra-c-util/>.
- *
- * Written by Russ Allbery <eagle@eyrie.org>
- *
- * The authors hereby relinquish any claim to any copyright that they may have
- * in this work, whether granted under contract or by operation of law or
- * international treaty, and hereby commit to the public, at large, that they
- * shall not, at any time in the future, seek to enforce any copyright in this
- * work against any person or entity, or prevent any person or entity from
- * copying, publishing, distributing or creating derivative works of this
- * work.
- */
-
-#include <config.h>
-#include <portable/system.h>
-
-/*
- * If we're running the test suite, rename strlcat to avoid conflicts with
- * the system version.
- */
-#if TESTING
-# undef strlcat
-# define strlcat test_strlcat
-size_t test_strlcat(char *, const char *, size_t);
-#endif
-
-size_t
-strlcat(char *dst, const char *src, size_t size)
-{
-    size_t used, length, copy;
-
-    used = strlen(dst);
-    length = strlen(src);
-    if (size > 0 && used < size - 1) {
-        copy = (length >= size - used) ? size - used - 1 : length;
-        memcpy(dst + used, src, copy);
-        dst[used + copy] = '\0';
-    }
-    return used + length;
-}
diff --git a/portable/strlcpy.c b/portable/strlcpy.c
deleted file mode 100644 (file)
index 60fdab3..0000000
+++ /dev/null
@@ -1,50 +0,0 @@
-/*
- * Replacement for a missing strlcpy.
- *
- * Provides the same functionality as the *BSD function strlcpy, originally
- * developed by Todd Miller and Theo de Raadt.  strlcpy works similarly to
- * strncpy, except saner and simpler.  The result is always nul-terminated
- * even if the source string is longer than the destination string, and the
- * total space required is returned.  The destination string is not nul-filled
- * like strncpy does, just nul-terminated.
- *
- * The canonical version of this file is maintained in the rra-c-util package,
- * which can be found at <http://www.eyrie.org/~eagle/software/rra-c-util/>.
- *
- * Written by Russ Allbery <eagle@eyrie.org>
- *
- * The authors hereby relinquish any claim to any copyright that they may have
- * in this work, whether granted under contract or by operation of law or
- * international treaty, and hereby commit to the public, at large, that they
- * shall not, at any time in the future, seek to enforce any copyright in this
- * work against any person or entity, or prevent any person or entity from
- * copying, publishing, distributing or creating derivative works of this
- * work.
- */
-
-#include <config.h>
-#include <portable/system.h>
-
-/*
- * If we're running the test suite, rename strlcpy to avoid conflicts with
- * the system version.
- */
-#if TESTING
-# undef strlcpy
-# define strlcpy test_strlcpy
-size_t test_strlcpy(char *, const char *, size_t);
-#endif
-
-size_t
-strlcpy(char *dst, const char *src, size_t size)
-{
-    size_t length, copy;
-
-    length = strlen(src);
-    if (size > 0) {
-        copy = (length >= size) ? size - 1 : length;
-        memcpy(dst, src, copy);
-        dst[copy] = '\0';
-    }
-    return length;
-}
index f78a4777977e8ff5cdbffc7e5e8b5fea09577235..76bd4ae6b6d2447b7af81e0d2a1aff443990fa29 100644 (file)
@@ -11,8 +11,6 @@ portable/asprintf
 portable/mkstemp
 portable/setenv
 portable/snprintf
-portable/strlcat
-portable/strlcpy
 server/admin
 server/backend
 server/keytab
diff --git a/tests/portable/strlcat-t.c b/tests/portable/strlcat-t.c
deleted file mode 100644 (file)
index 58aba58..0000000
+++ /dev/null
@@ -1,78 +0,0 @@
-/*
- * strlcat test suite.
- *
- * The canonical version of this file is maintained in the rra-c-util package,
- * which can be found at <http://www.eyrie.org/~eagle/software/rra-c-util/>.
- *
- * Written by Russ Allbery <eagle@eyrie.org>
- *
- * The authors hereby relinquish any claim to any copyright that they may have
- * in this work, whether granted under contract or by operation of law or
- * international treaty, and hereby commit to the public, at large, that they
- * shall not, at any time in the future, seek to enforce any copyright in this
- * work against any person or entity, or prevent any person or entity from
- * copying, publishing, distributing or creating derivative works of this
- * work.
- */
-
-#include <config.h>
-#include <portable/system.h>
-
-#include <tests/tap/basic.h>
-
-size_t test_strlcat(char *, const char *, size_t);
-
-
-int
-main(void)
-{
-    char buffer[10] = "";
-
-    plan(27);
-
-    is_int(3, test_strlcat(buffer, "foo", sizeof(buffer)),
-           "strlcat into empty buffer");
-    is_string("foo", buffer, "...with right output");
-    is_int(7, test_strlcat(buffer, " bar", sizeof(buffer)),
-           "...and append more");
-    is_string("foo bar", buffer, "...and output is still correct");
-    is_int(9, test_strlcat(buffer, "!!", sizeof(buffer)),
-           "...and append to buffer limit");
-    is_string("foo bar!!", buffer, "...output is still correct");
-    is_int(10, test_strlcat(buffer, "!", sizeof(buffer)),
-           "...append one more character");
-    is_string("foo bar!!", buffer, "...and output didn't change");
-    ok(buffer[9] == '\0', "...buffer still nul-terminated");
-    buffer[0] = '\0';
-    is_int(11, test_strlcat(buffer, "hello world", sizeof(buffer)),
-           "append single long string");
-    is_string("hello wor", buffer, "...string truncates properly");
-    ok(buffer[9] == '\0', "...buffer still nul-terminated");
-    buffer[0] = '\0';
-    is_int(7, test_strlcat(buffer, "sausage", 5), "lie about buffer length");
-    is_string("saus", buffer, "...contents are correct");
-    is_int(14, test_strlcat(buffer, "bacon eggs", sizeof(buffer)),
-           "...add more up to real size");
-    is_string("sausbacon", buffer, "...and result is correct");
-
-    /* Make sure that with a size of 0, the destination isn't changed. */
-    is_int(11, test_strlcat(buffer, "!!", 0), "no change with size of 0");
-    is_string("sausbacon", buffer, "...and content is the same");
-
-    /* Now play with empty strings. */
-    is_int(9, test_strlcat(buffer, "", 0),
-           "correct count when appending empty string");
-    is_string("sausbacon", buffer, "...and contents are unchanged");
-    buffer[0] = '\0';
-    is_int(0, test_strlcat(buffer, "", sizeof(buffer)),
-           "correct count when appending empty string to empty buffer");
-    is_string("", buffer, "...and buffer content is correct");
-    is_int(3, test_strlcat(buffer, "foo", 2), "append to length 2 buffer");
-    is_string("f", buffer, "...and got only a single character");
-    ok(buffer[1] == '\0', "...and buffer is still nul-terminated");
-    is_int(1, test_strlcat(buffer, "", sizeof(buffer)),
-           "append an empty string");
-    ok(buffer[1] == '\0', "...and buffer is still nul-terminated");
-
-    return 0;
-}
diff --git a/tests/portable/strlcat.c b/tests/portable/strlcat.c
deleted file mode 100644 (file)
index 8983bd8..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-#define TESTING 1
-#include <portable/strlcat.c>
diff --git a/tests/portable/strlcpy-t.c b/tests/portable/strlcpy-t.c
deleted file mode 100644 (file)
index 6652a7c..0000000
+++ /dev/null
@@ -1,70 +0,0 @@
-/*
- * strlcpy test suite.
- *
- * The canonical version of this file is maintained in the rra-c-util package,
- * which can be found at <http://www.eyrie.org/~eagle/software/rra-c-util/>.
- *
- * Written by Russ Allbery <eagle@eyrie.org>
- *
- * The authors hereby relinquish any claim to any copyright that they may have
- * in this work, whether granted under contract or by operation of law or
- * international treaty, and hereby commit to the public, at large, that they
- * shall not, at any time in the future, seek to enforce any copyright in this
- * work against any person or entity, or prevent any person or entity from
- * copying, publishing, distributing or creating derivative works of this
- * work.
- */
-
-#include <config.h>
-#include <portable/system.h>
-
-#include <tests/tap/basic.h>
-
-size_t test_strlcpy(char *, const char *, size_t);
-
-
-int
-main(void)
-{
-    char buffer[10];
-
-    plan(23);
-
-    is_int(3, test_strlcpy(buffer, "foo", sizeof(buffer)), "simple strlcpy");
-    is_string("foo", buffer, "...result is correct");
-    is_int(9, test_strlcpy(buffer, "hello wor", sizeof(buffer)),
-           "strlcpy exact length of buffer");
-    is_string("hello wor", buffer, "...result is correct");
-    is_int(10, test_strlcpy(buffer, "world hell", sizeof(buffer)),
-           "strlcpy one more than buffer length");
-    is_string("world hel", buffer, "...result is correct");
-    ok(buffer[9] == '\0', "...buffer is nul-terminated");
-    is_int(11, test_strlcpy(buffer, "hello world", sizeof(buffer)),
-           "strlcpy more than buffer length");
-    is_string("hello wor", buffer, "...result is correct");
-    ok(buffer[9] == '\0', "...buffer is nul-terminated");
-
-    /* Make sure that with a size of 0, the destination isn't changed. */
-    is_int(3, test_strlcpy(buffer, "foo", 0), "buffer unchanged if size 0");
-    is_string("hello wor", buffer, "...contents still the same");
-
-    /* Now play with empty strings. */
-    is_int(0, test_strlcpy(buffer, "", 0), "copy empty string with size 0");
-    is_string("hello wor", buffer, "...buffer unchanged");
-    is_int(0, test_strlcpy(buffer, "", sizeof(buffer)),
-           "copy empty string into full buffer");
-    is_string("", buffer, "...buffer now empty string");
-    is_int(3, test_strlcpy(buffer, "foo", 2),
-           "copy string into buffer of size 2");
-    is_string("f", buffer, "...got one character");
-    ok(buffer[1] == '\0', "...buffer is nul-terminated");
-    is_int(0, test_strlcpy(buffer, "", 1),
-           "copy empty string into buffer of size 1");
-    ok(buffer[0] == '\0', "...buffer is empty string");
-
-    /* Finally, check using strlcpy as strlen. */
-    is_int(3, test_strlcpy(NULL, "foo", 0), "use strlcpy as strlen");
-    is_int(11, test_strlcpy(NULL, "hello world", 0), "...again");
-
-    return 0;
-}
diff --git a/tests/portable/strlcpy.c b/tests/portable/strlcpy.c
deleted file mode 100644 (file)
index d444595..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-#define TESTING 1
-#include <portable/strlcpy.c>