diff options
author | Jeremy Allison <jra@samba.org> | 2005-03-07 23:54:24 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 10:55:54 -0500 |
commit | 84e7b7db8992db7812b8e09b8633c257657fb91c (patch) | |
tree | 2a06680c9fae5f9a904122cb07b08cfca536361f /source/client/clitar.c | |
parent | a42e3fc63cdbabd96f05e6af64344e1489ebf9da (diff) | |
download | samba-84e7b7db8992db7812b8e09b8633c257657fb91c.tar.gz samba-84e7b7db8992db7812b8e09b8633c257657fb91c.tar.xz samba-84e7b7db8992db7812b8e09b8633c257657fb91c.zip |
r5687: Fix for bug #2398 from Kevin Dalley <kevin@kelphead.org>.
smbtar shouldn't assume /dev/null means dryrun.
Jeremy.
Diffstat (limited to 'source/client/clitar.c')
-rw-r--r-- | source/client/clitar.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/source/client/clitar.c b/source/client/clitar.c index 14ebffb60ff..524feca1d2a 100644 --- a/source/client/clitar.c +++ b/source/client/clitar.c @@ -1761,11 +1761,7 @@ int tar_parseargs(int argc, char *argv[], const char *Optarg, int Optind) } } else { - if (tar_type=='c' && (dry_run || strcmp(argv[Optind], "/dev/null")==0)) { - if (!dry_run) { - DEBUG(0,("Output is /dev/null, assuming dry_run\n")); - dry_run = True; - } + if (tar_type=='c' && dry_run) { tarhandle=-1; } else if ((tar_type=='x' && (tarhandle = sys_open(argv[Optind], O_RDONLY, 0)) == -1) || (tar_type=='c' && (tarhandle=sys_creat(argv[Optind], 0644)) < 0)) { |