diff options
| author | nicm <nicm> | 2020-06-12 07:10:43 +0000 |
|---|---|---|
| committer | nicm <nicm> | 2020-06-12 07:10:43 +0000 |
| commit | cf63465eb08a658d1e7ec5fd1e1f70869117d3e9 (patch) | |
| tree | 5fb81711f0088c031158b8acd04c6ad8cfab7aed /arguments.c | |
| parent | 63c2ed14833bc543254b8dbd7d244e143a89b736 (diff) | |
Fix quoting with newlines and single quotes.
Diffstat (limited to 'arguments.c')
| -rw-r--r-- | arguments.c | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/arguments.c b/arguments.c index 0495aa0e..dd92a952 100644 --- a/arguments.c +++ b/arguments.c @@ -212,32 +212,35 @@ args_print(struct args *args) char * args_escape(const char *s) { - static const char quoted[] = " #\"';${}"; + static const char dquoted[] = " #';${}"; + static const char squoted[] = " \""; char *escaped, *result; - int flags; + int flags, quotes = 0; if (*s == '\0') { xasprintf(&result, "''"); return (result); } + if (s[strcspn(s, dquoted)] != '\0') + quotes = '"'; + else if (s[strcspn(s, squoted)] != '\0') + quotes = '\''; + if (s[0] != ' ' && - (strchr(quoted, s[0]) != NULL || s[0] == '~') && - s[1] == '\0') { + s[1] == '\0' && + (quotes != 0 || s[0] == '~')) { xasprintf(&escaped, "\\%c", s[0]); return (escaped); } - if (strchr(s, ' ') != NULL && strchr(s, '\'') == NULL) { - xasprintf(&escaped, "'%s'", s); - return (escaped); - } - flags = VIS_OCTAL|VIS_CSTYLE|VIS_TAB|VIS_NL; - if (s[strcspn(s, quoted)] != '\0') + if (quotes == '"') flags |= VIS_DQ; utf8_stravis(&escaped, s, flags); - if (flags & VIS_DQ) { + if (quotes == '\'') + xasprintf(&result, "'%s'", escaped); + else if (quotes == '"') { if (*escaped == '~') xasprintf(&result, "\"\\%s\"", escaped); else |