Fix O_APPEND Corruption
authorBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 9 Mar 2011 21:20:28 +0000 (13:20 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 9 Mar 2011 21:31:00 +0000 (13:31 -0800)
commitadf2e8778e66e6a749cec981da00463b342bd563
treede4ae9c8123fb78ab09156def7a45153e05bd0c9
parent17c37660a14cc5e26cc668fdb285933fee4b6cf4
Fix O_APPEND Corruption

Due to an uninitialized variable files opened with O_APPEND may
overwrite the start of the file rather than append to it.  This
was introduced accidentally when I removed the Solaris vnodes.

The zfs_range_lock_writer() function used to key off zf->z_vnode
to determine if a znode_t was for a zvol of zpl object.  With
the removal of vnodes this was replaced by the flag zp->z_is_zvol.
This flag was used to control the append behavior for range locks.

Unfortunately, this value was never properly initialized after
the vnode removal.  However, because most of memory is usually
zeros it happened to be set correctly most of the time making
the bug appear racy.  Properly initializing zp->z_is_zvol to
zero completely resolves the problem with O_APPEND.

Closes #126
module/zfs/zfs_znode.c