diff options
author | Rakesh Pandit <rakesh@tuxera.com> | 2014-04-19 22:05:41 +0300 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2014-05-02 17:03:45 +0200 |
commit | ad17d5608462e248c63f7be7b47bdbeaf971e52f (patch) | |
tree | d50fb61b35788298e6b4fa3969cb83939827f29a | |
parent | 6ca8f9cfc28364cf7378b54df41008bc52478f2e (diff) |
Btrfs-progs: make smatch checker happy (trivial fixes)
It complains errno never gets assigned to zero in find-root and since
errno anyway is zero at program started up, lets remove it.
Check "copy is less then zero" isn't possible because strtoull used by
arg_strtou64 wouldn't return -ve number.
Trivial space fixes.
Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
-rw-r--r-- | btrfs-find-root.c | 1 | ||||
-rw-r--r-- | btrfs-map-logical.c | 2 | ||||
-rw-r--r-- | cmds-check.c | 4 |
3 files changed, 2 insertions, 5 deletions
diff --git a/btrfs-find-root.c b/btrfs-find-root.c index db019233..25d79f1a 100644 --- a/btrfs-find-root.c +++ b/btrfs-find-root.c @@ -287,7 +287,6 @@ int main(int argc, char **argv) while ((opt = getopt(argc, argv, "l:o:g:")) != -1) { switch(opt) { - errno = 0; case 'o': search_objectid = arg_strtou64(optarg); break; diff --git a/btrfs-map-logical.c b/btrfs-map-logical.c index a907e4d4..e47a1ae1 100644 --- a/btrfs-map-logical.c +++ b/btrfs-map-logical.c @@ -153,8 +153,6 @@ int main(int ac, char **av) print_usage(); if (logical == 0) print_usage(); - if (copy < 0) - print_usage(); dev = av[optind]; diff --git a/cmds-check.c b/cmds-check.c index 3e089128..a290fe36 100644 --- a/cmds-check.c +++ b/cmds-check.c @@ -2882,7 +2882,7 @@ static int add_data_backref(struct cache_tree *extent_cache, u64 bytenr, } else { if (back->node.found_extent_tree) { fprintf(stderr, "Extent back ref already exists " - "for %llu parent %llu root %llu" + "for %llu parent %llu root %llu " "owner %llu offset %llu num_refs %lu\n", (unsigned long long)bytenr, (unsigned long long)parent, @@ -4735,7 +4735,7 @@ static int verify_backrefs(struct btrfs_trans_handle *trans, if (!best) { entry = find_entry(&entries, rec->start, rec->nr); if (!entry && (!broken_entries || !rec->found_rec)) { - fprintf(stderr, "Backrefs don't agree with eachother " + fprintf(stderr, "Backrefs don't agree with each other " "and extent record doesn't agree with anybody," " so we can't fix bytenr %Lu bytes %Lu\n", rec->start, rec->nr); |