Looking into printer driver issues, I ran across some peculiarities in
authorDavid Disseldorp <ddiss@suse.de>
Wed, 26 Jan 2011 22:32:01 +0000 (14:32 -0800)
committerJeremy Allison <jra@samba.org>
Wed, 26 Jan 2011 23:59:56 +0000 (00:59 +0100)
commiteb55338ccdc000feac92118553c5ce59bed87fa8
tree1e7cd26602e90569c5f41c1ba3a50c426c181a42
parentefde1d870b6a8c2d5461d9ae6cab5d3f2cbffb83
Looking into printer driver issues, I ran across some peculiarities in
copy_file():
- Firstly, if the source file is zero bytes, NT_STATUS_DISK_FULL is
  returned.
- Secondly, the conditional lseek is confusing. It fires when
  OPENX_FILE_EXISTS_OPEN is set and I can't see why the lseek is
  necessary in this case.
- Finally, the lseek error path also results in NT_STATUS_DISK_FULL.

Proposed fix for first and third point below.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Thu Jan 27 00:59:56 CET 2011 on sn-devel-104
source3/smbd/reply.c