re: Btrfs: fix up read_tree_block to return proper error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Liu Bo,

The patch 64c043de466d: "Btrfs: fix up read_tree_block to return
proper error" from May 25, 2015, leads to the following static
checker warning:

	fs/btrfs/relocation.c:1858 replace_path()
	error: 'eb' dereferencing possible ERR_PTR()

fs/btrfs/relocation.c
  1850                          eb = read_tree_block(dest, old_bytenr, old_ptr_gen);
  1851                          if (IS_ERR(eb)) {
                                    ^^^^^^^^^^

  1852                                  ret = PTR_ERR(eb);

Missing break or continue?

  1853                          } else if (!extent_buffer_uptodate(eb)) {
  1854                                  ret = -EIO;
  1855                                  free_extent_buffer(eb);
  1856                                  break;
  1857                          }
  1858                          btrfs_tree_lock(eb);
                                                ^^
Dereference.

  1859                          if (cow) {
  1860                                  ret = btrfs_cow_block(trans, dest, eb, parent,
  1861                                                        slot, &eb);

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux