upstream: make SSHBUF_DBG/SSHBUF_TELL (off by default and only enabled

via #define) dump to stderr rather than stdout

OpenBSD-Commit-ID: 10298513ee32db8390aecb0397d782d68cb14318

diff --git a/sshbuf-getput-basic.c b/sshbuf-getput-basic.c
index 9803fb5ed904c388c2a81f3c56d194c9fa632107..dcec0364b1def26a0145b53912d58d0af43d35e8 100644 (file)
--- a/sshbuf-getput-basic.c
+++ b/sshbuf-getput-basic.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sshbuf-getput-basic.c,v 1.11 2020/06/05 03:25:35 djm Exp $    */
+/*     $OpenBSD: sshbuf-getput-basic.c,v 1.12 2022/05/25 00:31:13 djm Exp $    */
 /*
  * Copyright (c) 2011 Damien Miller
  *
@@ -198,7 +198,7 @@ sshbuf_get_string(struct sshbuf *buf, u_char **valp, size_t *lenp)
                return r;
        if (valp != NULL) {
                if ((*valp = malloc(len + 1)) == NULL) {
-                       SSHBUF_DBG(("SSH_ERR_ALLOC_FAIL"));
+                       SSHBUF_DBG("SSH_ERR_ALLOC_FAIL");
                        return SSH_ERR_ALLOC_FAIL;
                }
                if (len != 0)
@@ -229,7 +229,7 @@ sshbuf_get_string_direct(struct sshbuf *buf, const u_char **valp, size_t *lenp)
                *lenp = len;
        if (sshbuf_consume(buf, len + 4) != 0) {
                /* Shouldn't happen */
-               SSHBUF_DBG(("SSH_ERR_INTERNAL_ERROR"));
+               SSHBUF_DBG("SSH_ERR_INTERNAL_ERROR");
                SSHBUF_ABORT();
                return SSH_ERR_INTERNAL_ERROR;
        }
@@ -248,16 +248,16 @@ sshbuf_peek_string_direct(const struct sshbuf *buf, const u_char **valp,
        if (lenp != NULL)
                *lenp = 0;
        if (sshbuf_len(buf) < 4) {
-               SSHBUF_DBG(("SSH_ERR_MESSAGE_INCOMPLETE"));
+               SSHBUF_DBG("SSH_ERR_MESSAGE_INCOMPLETE");
                return SSH_ERR_MESSAGE_INCOMPLETE;
        }
        len = PEEK_U32(p);
        if (len > SSHBUF_SIZE_MAX - 4) {
-               SSHBUF_DBG(("SSH_ERR_STRING_TOO_LARGE"));
+               SSHBUF_DBG("SSH_ERR_STRING_TOO_LARGE");
                return SSH_ERR_STRING_TOO_LARGE;
        }
        if (sshbuf_len(buf) - 4 < len) {
-               SSHBUF_DBG(("SSH_ERR_MESSAGE_INCOMPLETE"));
+               SSHBUF_DBG("SSH_ERR_MESSAGE_INCOMPLETE");
                return SSH_ERR_MESSAGE_INCOMPLETE;
        }
        if (valp != NULL)
@@ -283,14 +283,14 @@ sshbuf_get_cstring(struct sshbuf *buf, char **valp, size_t *lenp)
        /* Allow a \0 only at the end of the string */
        if (len > 0 &&
            (z = memchr(p , '\0', len)) != NULL && z < p + len - 1) {
-               SSHBUF_DBG(("SSH_ERR_INVALID_FORMAT"));
+               SSHBUF_DBG("SSH_ERR_INVALID_FORMAT");
                return SSH_ERR_INVALID_FORMAT;
        }
        if ((r = sshbuf_skip_string(buf)) != 0)
                return -1;
        if (valp != NULL) {
                if ((*valp = malloc(len + 1)) == NULL) {
-                       SSHBUF_DBG(("SSH_ERR_ALLOC_FAIL"));
+                       SSHBUF_DBG("SSH_ERR_ALLOC_FAIL");
                        return SSH_ERR_ALLOC_FAIL;
                }
                if (len != 0)
@@ -517,7 +517,7 @@ sshbuf_put_string(struct sshbuf *buf, const void *v, size_t len)
        int r;
 
        if (len > SSHBUF_SIZE_MAX - 4) {
-               SSHBUF_DBG(("SSH_ERR_NO_BUFFER_SPACE"));
+               SSHBUF_DBG("SSH_ERR_NO_BUFFER_SPACE");
                return SSH_ERR_NO_BUFFER_SPACE;
        }
        if ((r = sshbuf_reserve(buf, len + 4, &d)) < 0)
@@ -575,7 +575,7 @@ sshbuf_put_bignum2_bytes(struct sshbuf *buf, const void *v, size_t len)
        int r, prepend;
 
        if (len > SSHBUF_SIZE_MAX - 5) {
-               SSHBUF_DBG(("SSH_ERR_NO_BUFFER_SPACE"));
+               SSHBUF_DBG("SSH_ERR_NO_BUFFER_SPACE");
                return SSH_ERR_NO_BUFFER_SPACE;
        }
        /* Skip leading zero bytes */
@@ -625,7 +625,7 @@ sshbuf_get_bignum2_bytes_direct(struct sshbuf *buf,
                *lenp = len;
        if (sshbuf_consume(buf, olen + 4) != 0) {
                /* Shouldn't happen */
-               SSHBUF_DBG(("SSH_ERR_INTERNAL_ERROR"));
+               SSHBUF_DBG("SSH_ERR_INTERNAL_ERROR");
                SSHBUF_ABORT();
                return SSH_ERR_INTERNAL_ERROR;
        }

diff --git a/sshbuf-getput-crypto.c b/sshbuf-getput-crypto.c
index 2e61d3bcd809532058bf8e3d03aefab92ee2d7e4..0a72299184ec4d2b09366f7ef8b606f0a5a967cf 100644 (file)
--- a/sshbuf-getput-crypto.c
+++ b/sshbuf-getput-crypto.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sshbuf-getput-crypto.c,v 1.8 2019/11/15 06:00:20 djm Exp $    */
+/*     $OpenBSD: sshbuf-getput-crypto.c,v 1.9 2022/05/25 00:31:13 djm Exp $    */
 /*
  * Copyright (c) 2011 Damien Miller
  *
@@ -84,7 +84,7 @@ sshbuf_get_ec(struct sshbuf *buf, EC_POINT *v, const EC_GROUP *g)
        /* Skip string */
        if (sshbuf_get_string_direct(buf, NULL, NULL) != 0) {
                /* Shouldn't happen */
-               SSHBUF_DBG(("SSH_ERR_INTERNAL_ERROR"));
+               SSHBUF_DBG("SSH_ERR_INTERNAL_ERROR");
                SSHBUF_ABORT();
                return SSH_ERR_INTERNAL_ERROR;
        }
@@ -100,7 +100,7 @@ sshbuf_get_eckey(struct sshbuf *buf, EC_KEY *v)
        size_t len;
 
        if (pt == NULL) {
-               SSHBUF_DBG(("SSH_ERR_ALLOC_FAIL"));
+               SSHBUF_DBG("SSH_ERR_ALLOC_FAIL");
                return SSH_ERR_ALLOC_FAIL;
        }
        if ((r = sshbuf_peek_string_direct(buf, &d, &len)) < 0) {
@@ -119,7 +119,7 @@ sshbuf_get_eckey(struct sshbuf *buf, EC_KEY *v)
        /* Skip string */
        if (sshbuf_get_string_direct(buf, NULL, NULL) != 0) {
                /* Shouldn't happen */
-               SSHBUF_DBG(("SSH_ERR_INTERNAL_ERROR"));
+               SSHBUF_DBG("SSH_ERR_INTERNAL_ERROR");
                SSHBUF_ABORT();
                return SSH_ERR_INTERNAL_ERROR;
        }

diff --git a/sshbuf.c b/sshbuf.c
index 840b899b16beb0e87c558cf357ecff2b5d6ad98e..0e873ef1daaba4102e23f49e8e18f6ea295c2250 100644 (file)
--- a/sshbuf.c
+++ b/sshbuf.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sshbuf.c,v 1.16 2022/04/08 04:40:40 djm Exp $ */
+/*     $OpenBSD: sshbuf.c,v 1.17 2022/05/25 00:31:13 djm Exp $ */
 /*
  * Copyright (c) 2011 Damien Miller
  *
@@ -41,7 +41,7 @@ sshbuf_check_sanity(const struct sshbuf *buf)
            buf->size > buf->alloc ||
            buf->off > buf->size)) {
                /* Do not try to recover from corrupted buffer internals */
-               SSHBUF_DBG(("SSH_ERR_INTERNAL_ERROR"));
+               SSHBUF_DBG("SSH_ERR_INTERNAL_ERROR");
                ssh_signal(SIGSEGV, SIG_DFL);
                raise(SIGSEGV);
                return SSH_ERR_INTERNAL_ERROR;
@@ -52,7 +52,7 @@ sshbuf_check_sanity(const struct sshbuf *buf)
 static void
 sshbuf_maybe_pack(struct sshbuf *buf, int force)
 {
-       SSHBUF_DBG(("force %d", force));
+       SSHBUF_DBG("force %d", force);
        SSHBUF_TELL("pre-pack");
        if (buf->off == 0 || buf->readonly || buf->refcount > 1)
                return;
@@ -223,7 +223,7 @@ sshbuf_set_max_size(struct sshbuf *buf, size_t max_size)
        u_char *dp;
        int r;
 
-       SSHBUF_DBG(("set max buf = %p len = %zu", buf, max_size));
+       SSHBUF_DBG("set max buf = %p len = %zu", buf, max_size);
        if ((r = sshbuf_check_sanity(buf)) != 0)
                return r;
        if (max_size == buf->max_size)
@@ -241,7 +241,7 @@ sshbuf_set_max_size(struct sshbuf *buf, size_t max_size)
                        rlen = ROUNDUP(buf->size, SSHBUF_SIZE_INC);
                if (rlen > max_size)
                        rlen = max_size;
-               SSHBUF_DBG(("new alloc = %zu", rlen));
+               SSHBUF_DBG("new alloc = %zu", rlen);
                if ((dp = recallocarray(buf->d, buf->alloc, rlen, 1)) == NULL)
                        return SSH_ERR_ALLOC_FAIL;
                buf->cd = buf->d = dp;
@@ -309,7 +309,7 @@ sshbuf_allocate(struct sshbuf *buf, size_t len)
        u_char *dp;
        int r;
 
-       SSHBUF_DBG(("allocate buf = %p len = %zu", buf, len));
+       SSHBUF_DBG("allocate buf = %p len = %zu", buf, len);
        if ((r = sshbuf_check_reserve(buf, len)) != 0)
                return r;
        /*
@@ -327,12 +327,12 @@ sshbuf_allocate(struct sshbuf *buf, size_t len)
         */
        need = len + buf->size - buf->alloc;
        rlen = ROUNDUP(buf->alloc + need, SSHBUF_SIZE_INC);
-       SSHBUF_DBG(("need %zu initial rlen %zu", need, rlen));
+       SSHBUF_DBG("need %zu initial rlen %zu", need, rlen);
        if (rlen > buf->max_size)
                rlen = buf->alloc + need;
-       SSHBUF_DBG(("adjusted rlen %zu", rlen));
+       SSHBUF_DBG("adjusted rlen %zu", rlen);
        if ((dp = recallocarray(buf->d, buf->alloc, rlen, 1)) == NULL) {
-               SSHBUF_DBG(("realloc fail"));
+               SSHBUF_DBG("realloc fail");
                return SSH_ERR_ALLOC_FAIL;
        }
        buf->alloc = rlen;
@@ -354,7 +354,7 @@ sshbuf_reserve(struct sshbuf *buf, size_t len, u_char **dpp)
        if (dpp != NULL)
                *dpp = NULL;
 
-       SSHBUF_DBG(("reserve buf = %p len = %zu", buf, len));
+       SSHBUF_DBG("reserve buf = %p len = %zu", buf, len);
        if ((r = sshbuf_allocate(buf, len)) != 0)
                return r;
 
@@ -370,7 +370,7 @@ sshbuf_consume(struct sshbuf *buf, size_t len)
 {
        int r;
 
-       SSHBUF_DBG(("len = %zu", len));
+       SSHBUF_DBG("len = %zu", len);
        if ((r = sshbuf_check_sanity(buf)) != 0)
                return r;
        if (len == 0)
@@ -390,7 +390,7 @@ sshbuf_consume_end(struct sshbuf *buf, size_t len)
 {
        int r;
 
-       SSHBUF_DBG(("len = %zu", len));
+       SSHBUF_DBG("len = %zu", len);
        if ((r = sshbuf_check_sanity(buf)) != 0)
                return r;
        if (len == 0)

diff --git a/sshbuf.h b/sshbuf.h
index 07d54f0a988d3a517f5e0eae55d41f704c3de777..935302033da71e013ece7c721894d78bf03a7a36 100644 (file)
--- a/sshbuf.h
+++ b/sshbuf.h
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sshbuf.h,v 1.25 2022/01/22 00:43:43 djm Exp $ */
+/*     $OpenBSD: sshbuf.h,v 1.26 2022/05/25 00:31:13 djm Exp $ */
 /*
  * Copyright (c) 2011 Damien Miller
  *
@@ -395,16 +395,17 @@ u_int     sshbuf_refcount(const struct sshbuf *buf);
 
 # ifdef SSHBUF_DEBUG
 #  define SSHBUF_TELL(what) do { \
-               printf("%s:%d %s: %s size %zu alloc %zu off %zu max %zu\n", \
+               fprintf(stderr, \
+                   "%s:%d %s: %s size %zu alloc %zu off %zu max %zu\n", \
                    __FILE__, __LINE__, __func__, what, \
                    buf->size, buf->alloc, buf->off, buf->max_size); \
-               fflush(stdout); \
+               fflush(stderr); \
        } while (0)
-#  define SSHBUF_DBG(x) do { \
-               printf("%s:%d %s: ", __FILE__, __LINE__, __func__); \
-               printf x; \
-               printf("\n"); \
-               fflush(stdout); \
+#  define SSHBUF_DBG(...) do { \
+               fprintf(stderr, "%s:%d %s: ", __FILE__, __LINE__, __func__); \
+               fprintf(stderr, __VA_ARGS__); \
+               fprintf(stderr, "\n"); \
+               fflush(stderr); \
        } while (0)
 # else
 #  define SSHBUF_TELL(what)