Matt McCutchen's Web Site
/
rsync
/
rsync.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Improved the program that checks for broken large-file locking.
[rsync/rsync.git]
/
rsync.c
diff --git
a/rsync.c
b/rsync.c
index
f100491
..
1eab8ab
100644
(file)
--- a/
rsync.c
+++ b/
rsync.c
@@
-64,28
+64,28
@@
int delete_file(char *fname)
#else
ret = do_stat(fname, &st);
#endif
#else
ret = do_stat(fname, &st);
#endif
- if (ret)
{
+ if (ret)
return -1;
return -1;
- }
if (!S_ISDIR(st.st_mode)) {
if (!S_ISDIR(st.st_mode)) {
- if (robust_unlink(fname) == 0 || errno == ENOENT) return 0;
+ if (robust_unlink(fname) == 0 || errno == ENOENT)
+ return 0;
rprintf(FERROR, "delete_file: unlink %s failed: %s\n",
full_fname(fname), strerror(errno));
return -1;
}
rprintf(FERROR, "delete_file: unlink %s failed: %s\n",
full_fname(fname), strerror(errno));
return -1;
}
- if (do_rmdir(fname) == 0 || errno == ENOENT) return 0;
- if (!force_delete || !recurse ||
- (errno != ENOTEMPTY && errno != EEXIST)) {
+ if (do_rmdir(fname) == 0 || errno == ENOENT)
+ return 0;
+ if (!force_delete || !recurse
+ || (errno != ENOTEMPTY && errno != EEXIST)) {
rprintf(FERROR, "delete_file: rmdir %s failed: %s\n",
full_fname(fname), strerror(errno));
return -1;
}
/* now we do a recsursive delete on the directory ... */
rprintf(FERROR, "delete_file: rmdir %s failed: %s\n",
full_fname(fname), strerror(errno));
return -1;
}
/* now we do a recsursive delete on the directory ... */
- d = opendir(fname);
- if (!d) {
+ if (!(d = opendir(fname))) {
rprintf(FERROR, "delete_file: opendir %s failed: %s\n",
full_fname(fname), strerror(errno));
return -1;
rprintf(FERROR, "delete_file: opendir %s failed: %s\n",
full_fname(fname), strerror(errno));
return -1;
@@
-93,12
+93,12
@@
int delete_file(char *fname)
for (errno = 0, di = readdir(d); di; errno = 0, di = readdir(d)) {
char *dname = d_name(di);
for (errno = 0, di = readdir(d); di; errno = 0, di = readdir(d)) {
char *dname = d_name(di);
- if (
strcmp(dname,".") == 0
- ||
strcmp(dname,"..") == 0
)
+ if (
dname[0] == '.' && (dname[1] == '\0'
+ ||
(dname[1] == '.' && dname[2] == '\0'))
)
continue;
pathjoin(buf, sizeof buf, fname, dname);
if (verbose > 0)
continue;
pathjoin(buf, sizeof buf, fname, dname);
if (verbose > 0)
- rprintf(FINFO,"deleting %s\n", buf);
+ rprintf(FINFO,
"deleting %s\n", buf);
if (delete_file(buf) != 0) {
closedir(d);
return -1;
if (delete_file(buf) != 0) {
closedir(d);
return -1;
@@
-237,7
+237,7
@@
void finish_transfer(char *fname, char *fnametmp, struct file_struct *file)
/* move tmp file over real file */
ret = robust_rename(fnametmp, fname, file->mode & INITACCESSPERMS);
/* move tmp file over real file */
ret = robust_rename(fnametmp, fname, file->mode & INITACCESSPERMS);
- if (ret
!=
0) {
+ if (ret
<
0) {
rprintf(FERROR, "%s %s -> \"%s\": %s\n",
ret == -2 ? "copy" : "rename",
full_fname(fnametmp), fname, strerror(errno));
rprintf(FERROR, "%s %s -> \"%s\": %s\n",
ret == -2 ? "copy" : "rename",
full_fname(fnametmp), fname, strerror(errno));