[lkml]   [2003]   [May]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
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

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:


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.

Dave Olien


--- 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';
case 't':
value = '\t';
+ 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 '\\':
case '\'':
case '"':
+ 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
More majordomo info at
Please read the FAQ at
 \ /
  Last update: 2005-03-22 13:35    [W:0.036 / U:9.232 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site