=================================================================== RCS file: /cvsrepo/anoncvs/cvs/src/usr.bin/sed/compile.c,v retrieving revision 1.48 retrieving revision 1.49 diff -u -r1.48 -r1.49 --- src/usr.bin/sed/compile.c 2018/07/09 09:43:54 1.48 +++ src/usr.bin/sed/compile.c 2018/08/14 18:10:09 1.49 @@ -1,4 +1,4 @@ -/* $OpenBSD: compile.c,v 1.48 2018/07/09 09:43:54 schwarze Exp $ */ +/* $OpenBSD: compile.c,v 1.49 2018/08/14 18:10:09 schwarze Exp $ */ /*- * Copyright (c) 1992 Diomidis Spinellis. @@ -362,28 +362,40 @@ error(COMPILE, "\\ can not be used as a string delimiter"); else if (c == '\n') error(COMPILE, "newline can not be used as a string delimiter"); - while (*p) { - if (*p == '[' && *p != c) { - if ((d = compile_ccl(&p, d)) == NULL) - error(COMPILE, "unbalanced brackets ([])"); - continue; - } else if (*p == '\\' && p[1] == c) { - p++; - } else if (*p == '\\' && p[1] == '[') { - *d++ = *p++; - } else if (*p == '\\' && p[1] == 'n') { - *d++ = '\n'; - p += 2; - continue; - } else if (*p == '\\' && p[1] == '\\') { - *d++ = *p++; - } else if (*p == c) { + + while (p[0]) { + /* Unescaped delimiter: We are done. */ + if (p[0] == c) { *d = '\0'; - return (p + 1); + return p + 1; } - *d++ = *p++; + if (p[0] == '\\') { + /* Escaped delimiter: Skip the backslash. */ + if (p[1] == c) { + p++; + } else { + /* Backslash-n: Match linefeed. */ + if (p[1] == 'n') { + *d++ = '\n'; + p += 2; + /* Other escapes remain unchanged. */ + } else { + *d++ = *p++; + *d++ = *p++; + } + continue; + } + } + if (p[0] != '[') + *d++ = *p++; + /* + * Bracket expression: + * It may contain the delimiter without escaping. + */ + else if ((d = compile_ccl(&p, d)) == NULL) + error(COMPILE, "unbalanced brackets ([])"); } - return (NULL); + return NULL; }