lkml.org 
[lkml]   [2010]   [Apr]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip:perf/core] perf probe: Fix mis-estimation for shortening filename
    Commit-ID:  dd259c5db26ccda46409dbf6efc79d5a2b259e38
    Gitweb: http://git.kernel.org/tip/dd259c5db26ccda46409dbf6efc79d5a2b259e38
    Author: Masami Hiramatsu <mhiramat@redhat.com>
    AuthorDate: Wed, 14 Apr 2010 18:39:35 -0400
    Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
    CommitDate: Wed, 14 Apr 2010 17:41:14 -0300

    perf probe: Fix mis-estimation for shortening filename

    Fix mis-estimation size for making a short filename.
    Since the buffer size is 32 bytes and there are '@' prefix and
    '\0' termination, maximum shorten filename length should be
    30. This means, before searching '/', it should be 31 bytes.

    LKML-Reference: <20100414223935.14630.11954.stgit@localhost6.localdomain6>
    Signed-off-by: Masami Hiramatsu <mhiramat@redhat.com>
    Cc: Paul Mackerras <paulus@samba.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Mike Galbraith <efault@gmx.de>
    Cc: Frederic Weisbecker <fweisbec@gmail.com>
    Cc: Ingo Molnar <mingo@elte.hu>
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/util/probe-event.c | 4 ++--
    1 files changed, 2 insertions(+), 2 deletions(-)

    diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c
    index 1c4a20a..6d43839 100644
    --- a/tools/perf/util/probe-event.c
    +++ b/tools/perf/util/probe-event.c
    @@ -806,12 +806,12 @@ static char *synthesize_perf_probe_point(struct perf_probe_point *pp)
    goto error;
    }
    if (pp->file) {
    - len = strlen(pp->file) - 32;
    + len = strlen(pp->file) - 31;
    if (len < 0)
    len = 0;
    tmp = strchr(pp->file + len, '/');
    if (!tmp)
    - tmp = pp->file + len - 1;
    + tmp = pp->file + len;
    ret = e_snprintf(file, 32, "@%s", tmp + 1);
    if (ret <= 0)
    goto error;

    \
     
     \ /
      Last update: 2010-04-15 09:35    [W:3.218 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site