diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2008-02-05 01:27:21 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-05 12:44:07 -0500 |
commit | 59714d65dfbc86d5cb93adc5bac57a921cc2fa84 (patch) | |
tree | 10eda85ccab88fb707d6c6b4f4fb7f2e9bd1f4e1 /fs/exec.c | |
parent | 0ccf831cbee94df9c5006dd46248c0f07847dd7c (diff) |
get_task_comm(): return the result
It was dumb to make get_task_comm() return void. Change it to return a
pointer to the resulting output for caller convenience.
Cc: Ulrich Drepper <drepper@redhat.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Roland McGrath <roland@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/exec.c')
-rw-r--r-- | fs/exec.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -947,12 +947,13 @@ static void flush_old_files(struct files_struct * files) | |||
947 | spin_unlock(&files->file_lock); | 947 | spin_unlock(&files->file_lock); |
948 | } | 948 | } |
949 | 949 | ||
950 | void get_task_comm(char *buf, struct task_struct *tsk) | 950 | char *get_task_comm(char *buf, struct task_struct *tsk) |
951 | { | 951 | { |
952 | /* buf must be at least sizeof(tsk->comm) in size */ | 952 | /* buf must be at least sizeof(tsk->comm) in size */ |
953 | task_lock(tsk); | 953 | task_lock(tsk); |
954 | strncpy(buf, tsk->comm, sizeof(tsk->comm)); | 954 | strncpy(buf, tsk->comm, sizeof(tsk->comm)); |
955 | task_unlock(tsk); | 955 | task_unlock(tsk); |
956 | return buf; | ||
956 | } | 957 | } |
957 | 958 | ||
958 | void set_task_comm(struct task_struct *tsk, char *buf) | 959 | void set_task_comm(struct task_struct *tsk, char *buf) |