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 | ||