diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-03-25 20:52:00 +0100 |
|---|---|---|
| committer |  Robert C. Helling <helling@atdotde.de> | 2019-03-27 13:58:15 +0100 |
| commit | 643a964d099b2bac29500dcd70aaec3ee3e9b0bd (patch) | |
| tree | b57893ddf656996b20bcbd1ff89ebaf12c75180a /core/membuffer.c | |
| parent | 40a3e562b01de2827fa6c2c5fd9d46631f91dc36 (diff) | |
| download | subsurface-643a964d099b2bac29500dcd70aaec3ee3e9b0bd.tar.gz | |
Cleanup: unconstify string argument to add_to_string()
add_to_string() frees the original string that is passed in. This
should therefore not be of "const char *" type, as the contents
of the string *will* be modified (or more precisely: destroyed).
Same for the congener smtk_concat_str().
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/membuffer.c')
| -rw-r--r-- | core/membuffer.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/membuffer.c b/core/membuffer.c index 46cb541f1..71832a428 100644 --- a/core/membuffer.c +++ b/core/membuffer.c @@ -278,7 +278,7 @@ void put_quoted(struct membuffer *b, const char *text, int is_attribute, int is_ } } -char *add_to_string_va(const char *old, const char *fmt, va_list args) +char *add_to_string_va(char *old, const char *fmt, va_list args) { char *res; struct membuffer o = { 0 }, n = { 0 }; @@ -296,7 +296,7 @@ char *add_to_string_va(const char *old, const char *fmt, va_list args) * WARNING - this will free(old), the intended pattern is * string = add_to_string(string, fmt, ...) */ -char *add_to_string(const char *old, const char *fmt, ...) +char *add_to_string(char *old, const char *fmt, ...) { char *res; va_list args; |