freebsd-skq/sys/ufs
Tim J. Robbins 3632928957 Do not attempt to free NULL dinodes (i_din1 or i_din2) in ffs_ifree().
These fields can be left as NULL if ffs_vget() allocates an inode but
fails before the dinode memory has been allocated. There are two cases
when this can occur: when we lose a race and another process has added
the inode to the hash, and when reading the inode off disk fails.

The bug was observed by Kris on one of the package-building machines.
See http://marc.theaimsgroup.com/?l=freebsd-current&m=105172731013411&w=2
In Kris's case, it was the bread() that failed because of a disk error.

The alternative to this patch is to ensure that ffs_vget() does not call
vput() when the inode that hasn't been properly initialised.
2003-05-01 06:41:59 +00:00
..
ffs Do not attempt to free NULL dinodes (i_din1 or i_din2) in ffs_ifree(). 2003-05-01 06:41:59 +00:00
ufs Fix some easy, global, lint warnings. In most cases, this means 2003-04-30 12:57:40 +00:00