diff options
| author | Nicolas Iooss <nicolas.iooss_linux@m4x.org> | 2015-03-17 11:15:35 -0400 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2015-03-17 11:15:35 -0400 |
| commit | 5ce1aca814354c5942cdb5604e142a49be7033eb (patch) | |
| tree | e9931adea4fc3d1e967922733b16a383c80ffb5e /fs/reiserfs | |
| parent | 7dca0548a21e5efa445b68a73554ef863e09c623 (diff) | |
reiserfs: fix __RASSERT format string
__RASSERT format string does not use the PID argument. reiserfs_panic
arguments are therefore formatted with the wrong format specifier (for
example __LINE__ with %s). This bug was introduced when commit
c3a9c2109f84 ("reiserfs: rework reiserfs_panic") removed a
"reiserfs[%i]" prefix.
This bug is only triggered when using CONFIG_REISERFS_CHECK, otherwise
__RASSERT is never used.
Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
Acked-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/reiserfs')
| -rw-r--r-- | fs/reiserfs/reiserfs.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/reiserfs/reiserfs.h b/fs/reiserfs/reiserfs.h index bb79cddf0a1f..2adcde137c3f 100644 --- a/fs/reiserfs/reiserfs.h +++ b/fs/reiserfs/reiserfs.h | |||
| @@ -910,7 +910,6 @@ do { \ | |||
| 910 | if (!(cond)) \ | 910 | if (!(cond)) \ |
| 911 | reiserfs_panic(NULL, "assertion failure", "(" #cond ") at " \ | 911 | reiserfs_panic(NULL, "assertion failure", "(" #cond ") at " \ |
| 912 | __FILE__ ":%i:%s: " format "\n", \ | 912 | __FILE__ ":%i:%s: " format "\n", \ |
| 913 | in_interrupt() ? -1 : task_pid_nr(current), \ | ||
| 914 | __LINE__, __func__ , ##args); \ | 913 | __LINE__, __func__ , ##args); \ |
| 915 | } while (0) | 914 | } while (0) |
| 916 | 915 | ||
