diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2013-02-26 12:30:09 +0100 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2013-02-26 12:30:09 +0100 |
| commit | 905e8e0e2f0a1f51ab92424dcb005885ea8b2459 (patch) | |
| tree | aec55fed1263ba64e88ce59321d1c6760e91767a /util.c | |
| parent | 1d58fe9b3b925735bde72cac96a042b3cef03037 (diff) | |
| download | strace-905e8e0e2f0a1f51ab92424dcb005885ea8b2459.tar.gz strace-905e8e0e2f0a1f51ab92424dcb005885ea8b2459.tar.bz2 strace-905e8e0e2f0a1f51ab92424dcb005885ea8b2459.tar.xz | |
Mass replace error_msg("%s", "literal") -> error_msg("literal")
There is no need to print literal strings through "%s".
Only untrusted strings such as filenames need that.
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'util.c')
| -rw-r--r-- | util.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -808,9 +808,9 @@ umoven(struct tcb *tcp, long addr, int len, char *laddr) else if (errno != EINVAL && errno != ESRCH) /* EINVAL or ESRCH could be seen if process is gone, * all the rest is strange and should be reported. */ - perror_msg("%s", "process_vm_readv"); + perror_msg("process_vm_readv"); } else { - perror_msg("process_vm_readv: short read (%d < %d)", r, len); + error_msg("process_vm_readv: short read (%d < %d)", r, len); } } @@ -923,7 +923,7 @@ umovestr(struct tcb *tcp, long addr, int len, char *laddr) /* EINVAL or ESRCH could be seen * if process is gone, all the rest * is strange and should be reported. */ - perror_msg("%s", "process_vm_readv"); + perror_msg("process_vm_readv"); goto vm_readv_didnt_work; } if (memchr(local[0].iov_base, '\0', r)) |