dereferencing freed variable in "add basic DIO read/write support"

dereferencing freed variable in "add basic DIO read/write support"

[Dan Carpenter]

Hello,

Smatch complains about a possible freed pointer deref introduced by:
4b46fce2334 "Btrfs: add basic DIO read/write support".  Could you take a
look?

fs/btrfs/inode.c +5716 btrfs_submit_direct(79)
  5705  out_err:
  5706          kfree(dip->csums);
  5707          kfree(dip);
                ^^^^^^^^^^^

	dip is freed here.

  5708  free_ordered:
  5709          /*
  5710           * If this is a write, we need to clean up the reserved space and kill
  5711           * the ordered extent.
  5712           */
  5713          if (write) {
  5714                  struct btrfs_ordered_extent *ordered;
  5715                  ordered = btrfs_lookup_ordered_extent(inode,
  5716				dip->logical_offset);
                                ^^^^^^^^^^^^^^^^^^^

	dereferenced here.  Actually, dip could also be null here if the
	kmalloc failed()


regards,
dan carpenter

Re: dereferencing freed variable in "add basic DIO read/write support"

[Chris Mason]

On Fri, May 28, 2010 at 12:33:59PM +0200, Dan Carpenter wrote:
> Hello,
> 
> Smatch complains about a possible freed pointer deref introduced by:
> 4b46fce2334 "Btrfs: add basic DIO read/write support".  Could you take a
> look?

Thanks Dan, I'll patch this up.

-chris