The branch, master has been updated via 41d830e s3:smbd: remove unnecessary variable readret from read_file() via eb750d5 s3:smbd: remove ancient DMF_FIX from read_file(). from 80d3702 build: do not run fsrvp tests as part of selftest
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 41d830ea4fb8bf7cb20e08607ba9db9f53c03880 Author: Michael Adam <ob...@samba.org> Date: Fri Jun 8 16:06:24 2012 +0200 s3:smbd: remove unnecessary variable readret from read_file() Pair-Programmed-With: Volker Lendecke <v...@samba.org> Autobuild-User(master): Michael Adam <ob...@samba.org> Autobuild-Date(master): Fri Jun 8 18:08:09 CEST 2012 on sn-devel-104 commit eb750d54589542d9e6d9d48ff3bed1890cee7050 Author: Michael Adam <ob...@samba.org> Date: Fri Jun 8 14:23:09 2012 +0200 s3:smbd: remove ancient DMF_FIX from read_file(). Pair-Programmed-With: Volker Lendecke <v...@samba.org> ----------------------------------------------------------------------- Summary of changes: source3/smbd/fileio.c | 25 +++---------------------- 1 files changed, 3 insertions(+), 22 deletions(-) Changeset truncated at 500 lines: diff --git a/source3/smbd/fileio.c b/source3/smbd/fileio.c index ab505f4..a14be78 100644 --- a/source3/smbd/fileio.c +++ b/source3/smbd/fileio.c @@ -64,7 +64,7 @@ static bool read_from_write_cache(files_struct *fsp,char *data,off_t pos,size_t ssize_t read_file(files_struct *fsp,char *data,off_t pos,size_t n) { - ssize_t ret=0,readret; + ssize_t ret = 0; /* you can't read from print files */ if (fsp->print_file) { @@ -87,30 +87,11 @@ ssize_t read_file(files_struct *fsp,char *data,off_t pos,size_t n) fsp->fh->pos = pos; if (n > 0) { -#ifdef DMF_FIX - int numretries = 3; -tryagain: - readret = SMB_VFS_PREAD(fsp,data,n,pos); - - if (readret == -1) { - if ((errno == EAGAIN) && numretries) { - DEBUG(3,("read_file EAGAIN retry in 10 seconds\n")); - (void)sleep(10); - --numretries; - goto tryagain; - } - return -1; - } -#else /* NO DMF fix. */ - readret = SMB_VFS_PREAD(fsp,data,n,pos); + ret = SMB_VFS_PREAD(fsp,data,n,pos); - if (readret == -1) { + if (ret == -1) { return -1; } -#endif - if (readret > 0) { - ret += readret; - } } DEBUG(10,("read_file (%s): pos = %.0f, size = %lu, returned %lu\n", -- Samba Shared Repository