Wang Shilong
2014-Feb-19 11:17 UTC
[PATCH 0/4] Btrfs-progs: cleanups: new helper for parsing string to u64
There are many places that need parse string to u64 for btrfs commands, in fact, we do such things *too casually*, using atoi/atol/atoll..is not right at all, and even we don't check whether it is a valid string. Let's do everything more gracefully, we introduce a new helper btrfs_strtoull() which will do all the necessary checks.If we fail to parse string to u64, we will output message and exit directly, this is something like what usage() is doing. It is ok to not return erro to it's caller, because this function should be called when parsing arg (just like usage!) I convert most places to btrfs_strtoull, test patches with xfstests. Feel free to review and comment. Wang Shilong (4): Btrfs-progs: new helper to parse string to u64 for btrfs Btrfs-progs: switch to btrfs_strtoull() part1 Btrfs-progs: switch to btrfs_strtoull() part2 Btrfs-progs: switch to btrfs_strtoull() part3 btrfs-corrupt-block.c | 38 +++++++++----------------------------- btrfs-debug-tree.c | 2 +- btrfs-find-root.c | 23 +++-------------------- btrfs-image.c | 8 ++++---- btrfs-list.c | 14 +++----------- btrfs-map-logical.c | 26 ++++++-------------------- btrfs-show-super.c | 7 ++++--- btrfstune.c | 4 ++-- cmds-inspect.c | 8 ++++---- cmds-replace.c | 7 +------ cmds-restore.c | 20 ++++---------------- cmds-subvolume.c | 8 ++------ utils.c | 21 ++++++++++++++++++++- utils.h | 1 + 14 files changed, 64 insertions(+), 123 deletions(-) -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html