Fix zpool_read_label()
authorBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 9 Jul 2013 22:13:40 +0000 (15:13 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 9 Jul 2013 23:02:04 +0000 (16:02 -0700)
commitff3510c1a562914902d78f3d56552e126103f5a4
treef841ead1a0c30d85a0868231cd6baddc644ad96a
parent131cc95ca78b8552c13555d2cded791808008ff3
Fix zpool_read_label()

The zpool_read_label() function was subtly broken due to a
difference of behavior in fstat64(2) on Solaris vs Linux.

Under Solaris when a block device is stat'ed the st_size
field will contain the size of the device in bytes.  Under
Linux this is only true for regular file and symlinks.  A
compatibility function called fstat64_blk(2) was added
which can be used when the Solaris behavior is required.

This flaw was never noticed because the only time we would
need to use the device size is when the first two labels
are damaged.  I noticed this issue while adding the
zpool_clear_label() function which is similar in design
and does require us to write all the labels.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
lib/libzfs/libzfs_import.c