There is now a common function used to parse btrfs keys triplets so use that one. No functional changes.
Signed-off-by: Nikolay Borisov <nbori...@suse.com> --- btrfs-corrupt-block.c | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/btrfs-corrupt-block.c b/btrfs-corrupt-block.c index 4cc3528df105..7a5513c455e6 100644 --- a/btrfs-corrupt-block.c +++ b/btrfs-corrupt-block.c @@ -114,7 +114,7 @@ static void print_usage(int ret) printf("\t-i The inode item to corrupt (must also specify the field to corrupt)\n"); printf("\t-x The file extent item to corrupt (must also specify -i for the inode and -f for the field to corrupt)\n"); printf("\t-m The metadata block to corrupt (must also specify -f for the field to corrupt)\n"); - printf("\t-K The key to corrupt in the format <num>,<num>,<num> (must also specify -f for the field)\n"); + printf("\t-K <u64,u8,u64> Corrupt the given key (must also specify -f for the field)\n"); printf("\t-f The field in the item to corrupt\n"); printf("\t-I <u64,u8,u64> Corrupt an item corresponding to the passed key triplet (must also specify the field to corrupt and root for the item)\n"); printf("\t-D Corrupt a dir item, must specify key and field\n"); @@ -1129,6 +1129,7 @@ int main(int argc, char **argv) int corrupt_item = 0; int corrupt_di = 0; int delete = 0; + int should_corrupt_key = 0; u64 metadata_block = 0; u64 inode = 0; u64 file_extent = (u64)-1; @@ -1207,15 +1208,8 @@ int main(int argc, char **argv) metadata_block = arg_strtou64(optarg); break; case 'K': - ret = sscanf(optarg, "%llu,%u,%llu", - &key.objectid, - (unsigned int *)&key.type, - &key.offset); - if (ret != 3) { - fprintf(stderr, "error reading key " - "%d\n", errno); - print_usage(1); - } + should_corrupt_key = 1; + parse_key(&key.objectid, &key.type, &key.offset); break; case 'D': corrupt_di = 1; @@ -1370,7 +1364,7 @@ int main(int argc, char **argv) ret = delete_item(target, &key); goto out_close; } - if (key.objectid || key.offset || key.type) { + if (should_corrupt_key) { if (*field == 0) print_usage(1); ret = corrupt_key(root, &key, field); -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html