lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 20 Jul 2009 21:11:34 -0400 (EDT) From: Steven Rostedt <rostedt@...dmis.org> To: Jiri Olsa <jolsa@...hat.com> cc: Ingo Molnar <mingo@...e.hu>, lkml <linux-kernel@...r.kernel.org>, Li Zefan <lizf@...fujitsu.com>, Frederic Weisbecker <fweisbec@...il.com> Subject: Re: [PATCH] tracing: dont reset set_ftrace_filter/notrace when opened with r/w perm On Thu, 16 Jul 2009, Jiri Olsa wrote: > If user setup set_ftrace_filter/set_ftrace_notrace files and then opens them > with read&write permissions, the previous setup will be removed. This is exactly what it was suppose to do. man fopen: w+ Open for reading and writing. The file is created if it does not exist, otherwise it is truncated. The stream is positioned at the beginning of the file. Which means that if you open a file for "w+" it will truncate it. Hence, you will remove all previous settings. What you want is: a+ Open for reading and appending (writing at end of file). The file is created if it does not exist. The initial file position for reading is at the beginning of the file, but output is always appended to the end of the file. Change the belowe code from "w+" to "a+" and you get your expected result. Thus, the current code is correct. -- Steve > > Tested with following program: > > [snip] > #include <stdio.h> > #include <string.h> > > int main(int argc, char **argv) > { > FILE *f; > char *mode = "w+"; > char *file = argv[1]; > > if (argc == 3) > mode = argv[2]; > > if (NULL == (f = fopen(file, mode))) { > perror("fopen failed"); > return -1; > } > > while(!feof(f)) { > #define BUFLEN 100 > char buf[BUFLEN]; > memset(buf, 0, BUFLEN); > fgets(buf, BUFLEN, f); > printf(buf); > } > > fclose(f); > return 0; > } > [snip] > > wbr, > jirka > > > Signed-off-by: Jiri Olsa <jolsa@...hat.com> > > --- > diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c > index 4521c77..11394bc 100644 > --- a/kernel/trace/ftrace.c > +++ b/kernel/trace/ftrace.c > @@ -1661,10 +1661,6 @@ ftrace_regex_open(struct inode *inode, struct file *file, int enable) > return -ENOMEM; > > mutex_lock(&ftrace_regex_lock); > - if ((file->f_mode & FMODE_WRITE) && > - !(file->f_flags & O_APPEND)) > - ftrace_filter_reset(enable); > - > if (file->f_mode & FMODE_READ) { > iter->pg = ftrace_pages_start; > iter->flags = enable ? FTRACE_ITER_FILTER : > @@ -2260,6 +2256,9 @@ ftrace_regex_write(struct file *file, const char __user *ubuf, > return 0; > > mutex_lock(&ftrace_regex_lock); > + if ((file->f_mode & FMODE_WRITE) && > + !(file->f_flags & O_APPEND)) > + ftrace_filter_reset(enable); > > if (file->f_mode & FMODE_READ) { > struct seq_file *m = file->private_data; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists