[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] Workaround for Mac OS X 10.4 SMB bug

From: B. Wells <b_wells_at_appleisp.net>
Date: 2006-07-24 05:27:39 CEST

For this workaround, I followed the same pattern as the other Mac OS X
hack in the source file. A simpler patch would have been to prevent zero
byte write operations for all newly created files. But it may be that
such a write operation is needed on some file systems to properly create
the new file. Thus this patch compiles specificly for Mac OS X and only
alters functionality when creating files on SMB shares.

Of course, this patch is really useful when APR is also patched to
support file locking on SMB shares. I wrote such a patch several months
ago and posted it on my website.


I should clean it up and post it to the dev@apr.apache.org mailing list
to see if they are interested in it. Or perhaps even better would be to
redo it as a patch for svn_io_file_lock2 so that the patches stay with

Any suggestions?

Brian Wells

Workaround for Mac OS X 10.4 SMB bug that prevents creation of empty

This is a workaround for a Mac OS X 10.4 bug that results in an I/O
when an empty file is created by Subversion on an SMB share. This
happens when working with an FSFS-backed repository stored on a
Windows file

* subversion/libsvn_subr/io.c
   (svn_io_file_create): Prevent write operations of zero bytes for
Mac OS X
   when the file being created is on an SMB share.


Index: subversion/libsvn_subr/io.c
--- subversion/libsvn_subr/io.c (.../vendor/subversion/current)
(revision 31)
+++ subversion/libsvn_subr/io.c (.../trunk/subversion) (revision 31)
@@ -770,6 +770,29 @@
    return SVN_NO_ERROR;
+ Some of the most recent releases of Mac OS X 10.4 (10.4.7 for sure)
have a
+ bug that results in an I/O error when an attempt is made to perform
a write
+ operation of zero bytes to a file on a mounted SMB share. This is a
+ problem when Subversion calls svn_io_file_create() to create empty
+ This workaround will avoid write operations of zero bytes if the
file being
+ created is on an SMB share.
+ */
+#if defined(__APPLE__) && defined(__MACH__)
+#include <sys/param.h>
+#include <sys/mount.h>
(file,contents) \
+struct statfs
sfb; \
+if (strlen(contents) !=
0 \
+ || statfs(file, &sfb) ==
-1 \
+ || apr_strnatcmp(sfb.f_fstypename,"smbfs") != 0)
+#define MACOSX_SMBWRITE_HACK(file, contents) do {} while (0);
svn_error_t *svn_io_file_create (const char *file,
                                   const char *contents,
                                   apr_pool_t *pool)
@@ -781,6 +804,7 @@
                               (APR_WRITE | APR_CREATE | APR_EXCL),
+ MACOSX_SMBWRITE_HACK(file,contents)
    SVN_ERR (svn_io_file_write_full (f, contents, strlen (contents),
                                     &written, pool));
    SVN_ERR (svn_io_file_close (f, pool));

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 24 05:28:12 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.