fs/proc/array.c: convert to use string_escape_str()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 12 Feb 2015 23:01:11 +0000 (15:01 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 13 Feb 2015 02:54:12 +0000 (18:54 -0800)
Instead of custom approach let's use string_escape_str() to escape a given
string (task_name in this case).

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/proc/array.c

index bd117d0..a3ccf4c 100644 (file)
@@ -81,6 +81,7 @@
 #include <linux/pid_namespace.h>
 #include <linux/ptrace.h>
 #include <linux/tracehook.h>
+#include <linux/string_helpers.h>
 #include <linux/user_namespace.h>
 
 #include <asm/pgtable.h>
 
 static inline void task_name(struct seq_file *m, struct task_struct *p)
 {
-       int i;
-       char *buf, *end;
-       char *name;
+       char *buf;
        char tcomm[sizeof(p->comm)];
 
        get_task_comm(tcomm, p);
 
        seq_puts(m, "Name:\t");
-       end = m->buf + m->size;
        buf = m->buf + m->count;
-       name = tcomm;
-       i = sizeof(tcomm);
-       while (i && (buf < end)) {
-               unsigned char c = *name;
-               name++;
-               i--;
-               *buf = c;
-               if (!c)
-                       break;
-               if (c == '\\') {
-                       buf++;
-                       if (buf < end)
-                               *buf++ = c;
-                       continue;
-               }
-               if (c == '\n') {
-                       *buf++ = '\\';
-                       if (buf < end)
-                               *buf++ = 'n';
-                       continue;
-               }
-               buf++;
-       }
+
+       /* Ignore error for now */
+       string_escape_str(tcomm, &buf, m->size - m->count,
+                         ESCAPE_SPACE | ESCAPE_SPECIAL, "\n\\");
+
        m->count = buf - m->buf;
        seq_putc(m, '\n');
 }