Commit f4937fc7 authored by Joe Perches's avatar Joe Perches Committed by Luis Henriques

hpfs: hpfs_error: Remove static buffer, use vsprintf extension %pV instead

commit a28e4b2b upstream.

Removing unnecessary static buffers is good.
Use the vsprintf %pV extension instead.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarMikulas Patocka <mikulas@twibright.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent 5bbb9aca
...@@ -52,17 +52,20 @@ static void unmark_dirty(struct super_block *s) ...@@ -52,17 +52,20 @@ static void unmark_dirty(struct super_block *s)
} }
/* Filesystem error... */ /* Filesystem error... */
static char err_buf[1024];
void hpfs_error(struct super_block *s, const char *fmt, ...) void hpfs_error(struct super_block *s, const char *fmt, ...)
{ {
struct va_format vaf;
va_list args; va_list args;
va_start(args, fmt); va_start(args, fmt);
vsnprintf(err_buf, sizeof(err_buf), fmt, args);
vaf.fmt = fmt;
vaf.va = &args;
pr_err("filesystem error: %pV", &vaf);
va_end(args); va_end(args);
pr_err("filesystem error: %s", err_buf);
if (!hpfs_sb(s)->sb_was_error) { if (!hpfs_sb(s)->sb_was_error) {
if (hpfs_sb(s)->sb_err == 2) { if (hpfs_sb(s)->sb_err == 2) {
pr_cont("; crashing the system because you wanted it\n"); pr_cont("; crashing the system because you wanted it\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment