diff options
author | Martin Pool <mbp@samba.org> | 2001-12-02 22:47:30 +0000 |
---|---|---|
committer | Martin Pool <mbp@samba.org> | 2001-12-02 22:47:30 +0000 |
commit | 384958ed3dcd18e8ec7cf2e5a6142812f14229d8 (patch) | |
tree | 4d1d02f905cdd8ade0b519dd2e2b0e0b839652ec | |
parent | 1cd5beeb06e9df035aa36c89cdb9c3173a8e0bd6 (diff) | |
download | rsync-384958ed3dcd18e8ec7cf2e5a6142812f14229d8.tar.gz |
Note about crash from Ayamura KIKUCHI
-rw-r--r-- | batch.c | 4 |
1 files changed, 4 insertions, 0 deletions
@@ -150,6 +150,10 @@ void write_batch_argvs_file(int orig_argc, int argc, char **argv) /* Write argvs info to batch file */ for (i = argc - orig_argc; i < argc; i++) { + /* FIXME: This apparently crashes if rsync is run with + * just "rsync -F". I think directly manipulating + * argv[] is probably bogus -- what if -F is part of a + * run of several short options? */ if (!strcmp(argv[i], "-F")) { /* safer to change it here than script */ strncat(buff, "-f ", 3); /* chg to -f + ext to get ready for remote */ strncat(buff, batch_file_ext, |