From a84f97f7dbeaf1ad0b3537c10e409dcf77baec52 Mon Sep 17 00:00:00 2001
From: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Date: Sun, 23 May 2021 13:01:52 +1000
Subject: [PATCH 7/7] [grohtml]: Avoid deallocation of static strings.
* src/devices/grohtml/post-html.cpp (assert_state::add): Avoid potential
deallocation of statically-allocated strings. Use strsave() to
duplicate them so that they can be safely handed to a_delete(). Also
update diagnostic message to report name of complaining program
(continuing the long process of fixing Savannah #52463).
Fixes <https://savannah.gnu.org/bugs/index.php?60656>. Thanks to
Petru-Florin Mihancea for the report.
Also wrap source lines at 72 columns.
---
src/devices/grohtml/post-html.cpp | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/devices/grohtml/post-html.cpp b/src/devices/grohtml/post-html.cpp
index 7bc017e..5218f92 100644
--- a/src/devices/grohtml/post-html.cpp
+++ b/src/devices/grohtml/post-html.cpp
@@ -1737,13 +1737,14 @@ void assert_state::add (assert_pos **h,
}
if (v == NULL || v[0] != '=') {
if (f == NULL)
- f = "stdin";
+ f = strsave("stdin");
if (l == NULL)
- l = "<none>";
+ l = strsave("<none>");
if (v == NULL)
v = "no value at all";
- fprintf(stderr, "%s:%s:error in assert format of id=%s expecting value to be prefixed with an '=' got %s\n",
- f, l, i, v);
+ fprintf(stderr, "%s:%s:%s:error in assert format of id=%s;"
+ " expecting value to be prefixed with an '=', got %s\n",
+ program_name, f, l, i, v);
}
t->id = i;
t->val = v;
--
2.32.0