[groff] 31/82: [troff]: Catch `std::bad_alloc` exceptions (3/3).

[G. Branden Robinson]

gbranden pushed a commit to branch master
in repository groff.

commit c390daadefc502826338da055c7bd9ba9be79d1b
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
AuthorDate: Thu Jul 10 21:02:08 2025 -0500

    [troff]: Catch `std::bad_alloc` exceptions (3/3).
    
    ...from `new` operator.  Throw a fatal error indicating how much memory
    we couldn't allocate.
    
    * src/roff/troff/input.cpp (get_delimited_name): Do it.
    
    Exhibit:
    
    $ { printf '\\C@'; dd if=/dev/zero of=/dev/stdout bs=1M count=1024 \
        | tr '\0' 'a'; printf '@\n'; } | ./build/test-groff
    1024+0 records in
    1024+0 records out
    1073741824 bytes (1.1 GB, 1.0 GiB) copied, 10.1333 s, 106 MB/s
    troff:<standard input>:1: fatal error: cannot allocate 1073741824 bytes to 
read input line
---
 ChangeLog                |  2 +-
 src/roff/troff/input.cpp | 24 +++++++++++++++++++-----
 2 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index d1f5698c6..d5beb1727 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -5,7 +5,7 @@
        allocate.
 
        * src/roff/troff/input.cpp (read_long_escape_parameters)
-       (do_get_long_name): Do it.
+       (do_get_long_name, get_delimited_name): Do it.
 
 2025-07-10  G. Branden Robinson <g.branden.robin...@gmail.com>
 
diff --git a/src/roff/troff/input.cpp b/src/roff/troff/input.cpp
index c1d905a67..4bdebd4c5 100644
--- a/src/roff/troff/input.cpp
+++ b/src/roff/troff/input.cpp
@@ -5800,16 +5800,30 @@ static symbol get_delimited_name()
   }
   int start_level = input_stack::get_level();
   int buf_size = default_buffer_size;
-  char *buf = new char[buf_size];
+  char *buf = 0 /* nullptr */;
+  try {
+    // C++03: new char[buf_size]();
+    buf = new char[buf_size];
+  }
+  catch (const std::bad_alloc &e) {
+    fatal("cannot allocate %1 bytes to read input line", buf_size);
+  }
+  (void) memset(buf, 0, (buf_size * sizeof(char)));
   int i = 0;
   for (;;) {
     if ((i + 1) > buf_size) {
       char *old_buf = buf;
-      // C++03: new char[buf_size * 2]();
-      buf = new char[buf_size * 2];
-      (void) memset(buf, 0, (buf_size * 2 * sizeof(char)));
+      int new_buf_size = buf_size * 2;
+      // C++03: new char[new_buf_size]();
+      try {
+       buf = new char[new_buf_size];
+      }
+      catch (const std::bad_alloc &e) {
+       fatal("cannot allocate %1 bytes to read input line", buf_size);
+      }
+      (void) memset(buf, 0, (new_buf_size * sizeof(char)));
       memcpy(buf, old_buf, buf_size);
-      buf_size *= 2;
+      buf_size = new_buf_size;
       delete[] old_buf;
     }
     tok.next();

_______________________________________________
groff-commit mailing list
groff-commit@gnu.org
https://lists.gnu.org/mailman/listinfo/groff-commit