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 /smtk-import | |
| 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 'smtk-import')
| -rw-r--r-- | smtk-import/smartrak.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/smtk-import/smartrak.c b/smtk-import/smartrak.c index 63b3af799..8ef01de48 100644 --- a/smtk-import/smartrak.c +++ b/smtk-import/smartrak.c @@ -211,7 +211,7 @@ static MdbTableDef *smtk_open_table(MdbHandle *mdb, char *tablename, MdbColumn * This is based in add_to_string() and add_to_string_va(), and, as its parents * frees the original string. */ -static char *smtk_concat_str(const char *orig, const char *sep, const char *fmt, ...) +static char *smtk_concat_str(char *orig, const char *sep, const char *fmt, ...) { char *str; va_list args; @@ -229,7 +229,7 @@ static char *smtk_concat_str(const char *orig, const char *sep, const char *fmt, free_buffer(&out); free_buffer(&in); - free((void *)orig); + free(orig); return str; } |