Revert "tree-wide: use sizeof on static arrays"

This reverts commit 81a3bb64b4147ac6da3087cb77ac67828a2f2b76.

This commit broke all builders running with pid_max > 32768.

Reverting for now so we can bring the build farm back online.

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>

diff --git a/src/lxc/caps.c b/src/lxc/caps.c
index 8ca47bf66c00e906b85790fd96e89335ae5e9feb..c56fe732eb7b6912f3dc2605daa97fb1bc88b2c4 100644 (file)
--- a/src/lxc/caps.c
+++ b/src/lxc/caps.c
@@ -299,7 +299,7 @@ static long int _real_caps_last_cap(void)
                char buf[INTTYPE_TO_STRLEN(int)];
 
        again:
-               n = read(fd, buf, sizeof(buf));
+               n = read(fd, buf, INTTYPE_TO_STRLEN(int));
                if (n < 0 && errno == EINTR) {
                        goto again;
                } else if (n >= 0) {

diff --git a/src/lxc/cgroups/cgfsng.c b/src/lxc/cgroups/cgfsng.c
index 1fe561498bad95d3639fcb7ca26c0258c3516b7e..82eb46a3bfe14f3d71c78d848c92153a561fa31d 100644 (file)
--- a/src/lxc/cgroups/cgfsng.c
+++ b/src/lxc/cgroups/cgfsng.c
@@ -321,8 +321,8 @@ static char *lxc_cpumask_to_cpulist(uint32_t *bitarr, size_t nbits)
                if (!is_set(i, bitarr))
                        continue;
 
-               ret = snprintf(numstr, sizeof(numstr), "%zu", i);
-               if (ret < 0 || (size_t)ret >= sizeof(numstr)) {
+               ret = snprintf(numstr, INTTYPE_TO_STRLEN(size_t), "%zu", i);
+               if (ret < 0 || (size_t)ret >= INTTYPE_TO_STRLEN(size_t)) {
                        lxc_free_array((void **)cpulist, free);
                        return NULL;
                }

diff --git a/src/lxc/log.c b/src/lxc/log.c
index 2a7ee1eab923a2963c6d69738746112aa4b23250..38f7c889c394a9be87efdb4e52b508a75b9d6fde 100644 (file)
--- a/src/lxc/log.c
+++ b/src/lxc/log.c
@@ -247,8 +247,8 @@ static int lxc_unix_epoch_to_utc(char *buf, size_t bufsize, const struct timespe
        seconds = (time->tv_sec - d_in_s - h_in_s - (minutes * 60));
 
        /* Make string from nanoseconds. */
-       ret = snprintf(nanosec, sizeof(nanosec), "%"PRId64, (int64_t)time->tv_nsec);
-       if (ret < 0 || ret >= sizeof(nanosec))
+       ret = snprintf(nanosec, INTTYPE_TO_STRLEN(int64_t), "%"PRId64, (int64_t)time->tv_nsec);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(int64_t))
                return -1;
 
        /* Create final timestamp for the log and shorten nanoseconds to 3

diff --git a/src/lxc/lxccontainer.c b/src/lxc/lxccontainer.c
index cdd1642026949da000e2a394e0d2d77d1125faff..f0f81724f1d772ac81cc1244a8e6a2172094f25e 100644 (file)
--- a/src/lxc/lxccontainer.c
+++ b/src/lxc/lxccontainer.c
@@ -1039,8 +1039,8 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a
                int ret, w;
                char pidstr[INTTYPE_TO_STRLEN(int)];
 
-               w = snprintf(pidstr, sizeof(pidstr), "%d", (int)lxc_raw_getpid());
-               if (w < 0 || (size_t)w >= sizeof(pidstr)) {
+               w = snprintf(pidstr, INTTYPE_TO_STRLEN(int), "%d", (int)lxc_raw_getpid());
+               if (w < 0 || (size_t)w >= INTTYPE_TO_STRLEN(int)) {
                        free_init_cmd(init_cmd);
                        lxc_free_handler(handler);
 

diff --git a/src/lxc/monitor.c b/src/lxc/monitor.c
index 4cca9002ce820db04ef8ba16e627671574812c91..cdf15b659cd102cb8cde5fc0ea63732c409b8239 100644 (file)
--- a/src/lxc/monitor.c
+++ b/src/lxc/monitor.c
@@ -371,8 +371,8 @@ int lxc_monitord_spawn(const char *lxcpath)
 
        close(pipefd[0]);
 
-       ret = snprintf(pipefd_str, sizeof(pipefd_str), "%d", pipefd[1]);
-       if (ret < 0 || ret >= sizeof(pipefd_str)) {
+       ret = snprintf(pipefd_str, INTTYPE_TO_STRLEN(int), "%d", pipefd[1]);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(int)) {
                ERROR("Failed to create pid argument to pass to monitord.");
                _exit(EXIT_FAILURE);
        }

diff --git a/src/lxc/tools/lxc_monitor.c b/src/lxc/tools/lxc_monitor.c
index 86a77c3afe38cf3a0d4e2d41745b6e3bcedc59b7..5e296082e6e6a7cb34b0eea4c4193fc67448d397 100644 (file)
--- a/src/lxc/tools/lxc_monitor.c
+++ b/src/lxc/tools/lxc_monitor.c
@@ -224,8 +224,8 @@ static int lxc_tool_monitord_spawn(const char *lxcpath)
 
        close(pipefd[0]);
 
-       ret = snprintf(pipefd_str, sizeof(pipefd_str), "%d", pipefd[1]);
-       if (ret < 0 || ret >= sizeof(pipefd_str)) {
+       ret = snprintf(pipefd_str, INTTYPE_TO_STRLEN(int), "%d", pipefd[1]);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(int)) {
                ERROR("Failed to create pid argument to pass to monitord");
                _exit(EXIT_FAILURE);
        }

diff --git a/src/lxc/utils.c b/src/lxc/utils.c
index 77ad76b498b2edc5548612009c1644df9d29bc8e..1f0ba897187345ec0f866b421718534eacf16339 100644 (file)
--- a/src/lxc/utils.c
+++ b/src/lxc/utils.c
@@ -1165,7 +1165,7 @@ int lxc_mount_proc_if_needed(const char *rootfs)
                return -1;
        }
 
-       linklen = readlink(path, link, sizeof(link));
+       linklen = readlink(path, link, INTTYPE_TO_STRLEN(pid_t));
 
        ret = snprintf(path, MAXPATHLEN, "%s/proc", rootfs);
        if (ret < 0 || ret >= MAXPATHLEN) {
@@ -1179,7 +1179,7 @@ int lxc_mount_proc_if_needed(const char *rootfs)
                        return -1;
 
                goto domount;
-       } else if (linklen >= sizeof(link)) {
+       } else if (linklen >= INTTYPE_TO_STRLEN(pid_t)) {
                link[linklen - 1] = '\0';
                ERROR("readlink returned truncated content: \"%s\"", link);
                return -1;

diff --git a/src/tests/lxc-test-utils.c b/src/tests/lxc-test-utils.c
index d0c07dc59dd65f905d4de94d5776bda8695998cb..20f71b9a564ffa1ac82f8bbc79ef22262ebc92f2 100644 (file)
--- a/src/tests/lxc-test-utils.c
+++ b/src/tests/lxc-test-utils.c
@@ -252,14 +252,14 @@ void test_lxc_safe_uint(void)
        lxc_test_assert_abort((-EINVAL == lxc_safe_uint("    -123", &n)));
        lxc_test_assert_abort((-EINVAL == lxc_safe_uint("-123", &n)));
 
-       ret = snprintf(numstr, sizeof(numstr), "%" PRIu64, (uint64_t)UINT_MAX);
-       if (ret < 0 || ret >= sizeof(numstr))
+       ret = snprintf(numstr, INTTYPE_TO_STRLEN(uint64_t), "%" PRIu64, (uint64_t)UINT_MAX);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(uint64_t))
                exit(EXIT_FAILURE);
 
        lxc_test_assert_abort((0 == lxc_safe_uint(numstr, &n)) && n == UINT_MAX);
 
-       ret = snprintf(numstr, sizeof(numstr), "%" PRIu64, (uint64_t)UINT_MAX + 1);
-       if (ret < 0 || ret >= sizeof(numstr))
+       ret = snprintf(numstr, INTTYPE_TO_STRLEN(uint64_t), "%" PRIu64, (uint64_t)UINT_MAX + 1);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(uint64_t))
                exit(EXIT_FAILURE);
 
        lxc_test_assert_abort((-ERANGE == lxc_safe_uint(numstr, &n)));
@@ -285,26 +285,26 @@ void test_lxc_safe_int(void)
        signed int n;
        char numstr[INTTYPE_TO_STRLEN(uint64_t)];
 
-       ret = snprintf(numstr, sizeof(numstr), "%" PRIu64, (uint64_t)INT_MAX);
-       if (ret < 0 || ret >= sizeof(numstr))
+       ret = snprintf(numstr, INTTYPE_TO_STRLEN(uint64_t), "%" PRIu64, (uint64_t)INT_MAX);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(uint64_t))
                exit(EXIT_FAILURE);
 
        lxc_test_assert_abort((0 == lxc_safe_int(numstr, &n)) && n == INT_MAX);
 
-       ret = snprintf(numstr, sizeof(numstr), "%" PRIu64, (uint64_t)INT_MAX + 1);
-       if (ret < 0 || ret >= sizeof(numstr))
+       ret = snprintf(numstr, INTTYPE_TO_STRLEN(uint64_t), "%" PRIu64, (uint64_t)INT_MAX + 1);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(uint64_t))
                exit(EXIT_FAILURE);
 
        lxc_test_assert_abort((-ERANGE == lxc_safe_int(numstr, &n)));
 
-       ret = snprintf(numstr, sizeof(numstr), "%" PRId64, (int64_t)INT_MIN);
-       if (ret < 0 || ret >= sizeof(numstr))
+       ret = snprintf(numstr, INTTYPE_TO_STRLEN(int64_t), "%" PRId64, (int64_t)INT_MIN);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(int64_t))
                exit(EXIT_FAILURE);
 
        lxc_test_assert_abort((0 == lxc_safe_int(numstr, &n)) && n == INT_MIN);
 
-       ret = snprintf(numstr, sizeof(numstr), "%" PRId64, (int64_t)INT_MIN - 1);
-       if (ret < 0 || ret >= sizeof(numstr))
+       ret = snprintf(numstr, INTTYPE_TO_STRLEN(int64_t), "%" PRId64, (int64_t)INT_MIN - 1);
+       if (ret < 0 || ret >= INTTYPE_TO_STRLEN(int64_t))
                exit(EXIT_FAILURE);
 
        lxc_test_assert_abort((-ERANGE == lxc_safe_int(numstr, &n)));