Whamcloud - gitweb
debugfs: restore and tweak original error messaging
authorEric Whitney <enwlinux@gmail.com>
Mon, 30 Dec 2013 21:43:46 +0000 (16:43 -0500)
committerTheodore Ts'o <tytso@mit.edu>
Mon, 30 Dec 2013 21:56:19 +0000 (16:56 -0500)
commita25fffae5cbdb9349fa3c358c9714584e9de7b33
treeeeb5bbc9f2bc10984a7ea6462eef9ff34f080342
parentd7e6e9787113db16df4c6c50a92da97364149e1a
debugfs: restore and tweak original error messaging

In response to reviewer comments, commit fe56188b07 included changes
that modified some of the code used to output error messages when
checking user-supplied block numbers.  These changes converted calls
to parse_ulonglong() to calls to strtoblk().  Because strtoblk() calls
parse_ulonglong(), and both output error messages, two redundant and
relatively generic messages were output on each error.

Fix this by removing the error message output from strtoblk(), and
extending it to accept an optional error message argument that it
supplies in lieu of a default to parse_ulonglong().  Also, revert to
the more descriptive original error messages with mods per reviewer
comments, and fix an error message in do_replace_node().

Signed-off-by: Eric Whitney <enwlinux@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
debugfs/debugfs.c
debugfs/debugfs.h
debugfs/extent_inode.c
debugfs/icheck.c
debugfs/util.c