don't return 0 on error - bad for scripts
authorHerb Lewis <hlewis@chomps.localdomain>
Fri, 12 Dec 2008 21:47:10 +0000 (13:47 -0800)
committerHerb Lewis <hlewis@chomps.localdomain>
Fri, 12 Dec 2008 21:47:10 +0000 (13:47 -0800)
source/utils/smbcacls.c

index 06ec984e012015b6f5e3ee011884e2a136492d6c..611df8a5cf68fdb485def607824bb5170bbd3f25 100644 (file)
@@ -737,6 +737,8 @@ static int owner_set(struct cli_state *cli, enum chown_mode change_mode,
 
        if (!cli_set_secdesc(cli, fnum, sd)) {
                printf("ERROR: secdesc set failed: %s\n", cli_errstr(cli));
+               cli_close(cli, fnum);
+               return EXIT_FAILED;
        }
 
        cli_close(cli, fnum);