Messages in this thread Patch in this message |  | | | Date | Thu, 29 May 2003 15:36:03 -0700 | | From | Dave Olien <> | | Subject | [PATCH] sparse tool handling of escaped characters in strings |
| |
Hi, Linus. I'm just beginning to look over your sparse tool library. I decided to start with something simple. Running your check tool on the 2.5.70 kernel produces 25 warnings of the forms:
warning: drivers/char/n_tty.c:198:35: Unknown escape 'r' warning: drivers/char/n_tty.c:225:11: Unknown escape 'b' warning: drivers/char/n_tty.c:547:17: Unknown escape 'a'
They come from character strings that contain escaped characters that escapechar() in tokenize.c didn't recognize. I added cases for these characters.
The warning below is a little more interesting.
warning: fs/proc/array.c:338:1: Unknown escape ' '
It comes from a string that is continued on multiple lines, of the form:
printf("abcdef\ ghijkl");
This is different from the other cases because the \<newline> character should never appear in the parsed string.
As a first fix, I've made escapechar() call itself recursively. One flaw with this fix is that escapechar() will mark the first character on the second line of the input string as "escaped":
/* Mark it as escaped */ value |= 0x100; But, none of the callers to escapechar() ever use or preserve this information. Can this "escaped" character information be removed.
Or, I could add a flag (ugh) that causes the code to NOT add the escape flag in this particular case.
Or, I could find a different solution to this case.
Thanks! Dave Olien OSDL
-------------------------------------------------------------------------- --- sparse_original/tokenize.c 2003-05-29 14:22:20.000000000 -0700 +++ sparse_test/tokenize.c 2003-05-29 15:19:25.000000000 -0700 @@ -340,18 +340,39 @@ next = nextchar(stream); if (value != type) { switch (value) { - case 'n': - value = '\n'; + case 'a': + value = '\a'; + break; + case 'b': + value = '\b'; break; case 't': value = '\t'; break; + case 'n': + value = '\n'; + break; + case 'v': + value = '\v'; + break; + case 'f': + value = '\f'; + break; + case 'r': + value = '\r'; + break; + case 'e': + value = '\e'; + break; case '\\': break; case '\'': break; case '"': break; + case '\n': + next = escapechar(next, type, stream, &value); + break; case '0'...'7': { int nr = 2; value -= '0'; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |