[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] bug_occurred: a place where we assumed that "buf" was still a buffer



commit 809a3a748d5f0cbaa0357ec63388ed291fb7b0bd
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Sat Mar 30 21:07:15 2019 -0400

    bug_occurred: a place where we assumed that "buf" was still a buffer
    
    In 9c132a5f9e87a2cd0f we replaced "buf" with a pointer and replaced
    one instance of snprintf with asprintf -- but there was still one
    snprintf left over, being crashy.
    
    Fixes bug 29967; bug not in any released Tor. This is CID 1444262.
---
 src/lib/log/util_bug.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/log/util_bug.c b/src/lib/log/util_bug.c
index b9dc296fc..65ab7bc9c 100644
--- a/src/lib/log/util_bug.c
+++ b/src/lib/log/util_bug.c
@@ -117,7 +117,7 @@ tor_bug_occurred_(const char *fname, unsigned int line,
     }
     log_warn(LD_BUG, "%s:%u: %s: This line should not have been reached.%s",
              fname, line, func, once_str);
-    tor_snprintf(buf, sizeof(buf),
+    tor_asprintf(&buf,
                  "Line unexpectedly reached at %s at %s:%u",
                  func, fname, line);
   } else {



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits