[prev in list] [next in list] [prev in thread] [next in thread]
List: rsync-cvs
Subject: [SCM] The rsync repository. - branch master updated
From: Rsync CVS commit messages <rsync-cvs () lists ! samba ! org>
Date: 2011-04-05 20:37:38
Message-ID: mailman.110.1302036006.1434.rsync-cvs () lists ! samba ! org
[Download RAW message or body]
The branch, master has been updated
via b223d96 Add some temp-name dot heuristics for OS X's sake.
from 28b519c Applying the preallocate patch.
;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit b223d96bf0d040cfa445a45213fc517ab8389785
Author: Wayne Davison <wayned@samba.org>
Date: Tue Apr 5 13:35:13 2011 -0700
Add some temp-name dot heuristics for OS X's sake.
- Drop one leading '.' from the filename (before adding our own).
- Drop one trailing '.' from a (possibly truncated) name prior to
the .XXXXXX suffix being added.
- Allow the temp-name to collapse to just the .XXXXXX suffix
if the path is long enough to require that.
Note that we don't try to remove multiple dots from a filename
that actually has multiple consecutive dots, since we might as
well learn early if the final name is going to fail or not.
-----------------------------------------------------------------------
Summary of changes:
receiver.c | 32 ++++++++++++++++++++------------
1 files changed, 20 insertions(+), 12 deletions(-)
Changeset truncated at 500 lines:
diff --git a/receiver.c b/receiver.c
index 4c43bc1..1819830 100644
--- a/receiver.c
+++ b/receiver.c
@@ -94,7 +94,7 @@ static int updating_basis_or_equiv;
* transfer is in progress. */
int get_tmpname(char *fnametmp, const char *fname, BOOL make_unique)
{
- int maxname, added, length = 0;
+ int maxname, length = 0;
const char *f;
char *suf;
@@ -113,6 +113,8 @@ int get_tmpname(char *fnametmp, const char *fname, BOOL make_unique)
}
} else
f = fname;
+ if (*f == '.') /* avoid an extra leading dot for OS X's sake */
+ f++;
fnametmp[length++] = '.';
/* The maxname value is bufsize, and includes space for the '\0'.
@@ -120,24 +122,30 @@ int get_tmpname(char *fnametmp, const char *fname, BOOL make_unique)
maxname = MIN(MAXPATHLEN - length - TMPNAME_SUFFIX_LEN,
NAME_MAX - 1 - TMPNAME_SUFFIX_LEN);
- if (maxname < 1) {
+ if (maxname < 0) {
rprintf(FERROR_XFER, "temporary filename too long: %s\n", fname);
fnametmp[0] = '\0';
return 0;
}
- added = strlcpy(fnametmp + length, f, maxname);
- if (added >= maxname)
- added = maxname - 1;
- suf = fnametmp + length + added;
-
- /* Trim any dangling high-bit chars if the first-trimmed char (if any) is
- * also a high-bit char, just in case we cut into a multi-byte sequence.
- * We are guaranteed to stop because of the leading '.' we added. */
- if ((int)f[added] & 0x80) {
- while ((int)suf[-1] & 0x80)
+ if (maxname) {
+ int added = strlcpy(fnametmp + length, f, maxname);
+ if (added >= maxname)
+ added = maxname - 1;
+ suf = fnametmp + length + added;
+
+ /* Trim any dangling high-bit chars if the first-trimmed char (if any) is
+ * also a high-bit char, just in case we cut into a multi-byte sequence.
+ * We are guaranteed to stop because of the leading '.' we added. */
+ if ((int)f[added] & 0x80) {
+ while ((int)suf[-1] & 0x80)
+ suf--;
+ }
+ /* trim one trailing dot before our suffix's dot */
+ if (suf[-1] == '.')
suf--;
- }
+ } else
+ suf = fnametmp + length - 1; /* overwrite the leading dot with suffix's dot */
if (make_unique) {
static unsigned counter_limit;
--
The rsync repository.
_______________________________________________
rsync-cvs mailing list
rsync-cvs@lists.samba.org
https://lists.samba.org/mailman/listinfo/rsync-cvs
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic