Skip to content

Commit

Permalink
perf_counter: fix off task->comm by one
Browse files Browse the repository at this point in the history
strlen() does not include the \0.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
  • Loading branch information
Ingo Molnar committed Apr 9, 2009
1 parent ca8f2d7 commit 888fcee
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions kernel/perf_counter.c
Original file line number Diff line number Diff line change
Expand Up @@ -1989,7 +1989,7 @@ static void perf_counter_comm_event(struct perf_comm_event *comm_event)
unsigned int size;
char *comm = comm_event->task->comm;

size = ALIGN(strlen(comm), sizeof(u64));
size = ALIGN(strlen(comm)+1, sizeof(u64));

comm_event->comm = comm;
comm_event->comm_size = size;
Expand Down Expand Up @@ -2109,7 +2109,7 @@ static void perf_counter_mmap_event(struct perf_mmap_event *mmap_event)
}

got_name:
size = ALIGN(strlen(name), sizeof(u64));
size = ALIGN(strlen(name)+1, sizeof(u64));

mmap_event->file_name = name;
mmap_event->file_size = size;
Expand Down

0 comments on commit 888fcee

Please sign in to comment.