diff options
| author | Andreas Schneider <asn@samba.org> | 2015-07-17 09:35:11 +0200 |
|---|---|---|
| committer | Andreas Schneider <asn@cryptomilk.org> | 2015-07-17 19:07:23 +0200 |
| commit | 48a4d5a4078ff2a66dd753323d6e5d76d34b9828 (patch) | |
| tree | e023ed6e31b0a6a81d14e1788d7fb20ed6bc4d8e /source3/smbd | |
| parent | 323e4f89fca2f4f0e017a36a27427be32b42fc54 (diff) | |
| download | samba-48a4d5a4078ff2a66dd753323d6e5d76d34b9828.tar.gz | |
s3-smbd: Leave sys_disk_free() if dfree command is used
If we have a broken system which reports incorrect sizes we provide the
'dfree command'. This command makes sure Samba gets the correct values.
However after that we call the quota command which then reports the
broken values. The dfree command should take care to provide the correct
values and in case of quota's it should also calculate the quote
correctly.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11403
Pair-Programmed-With: Michael Adam <obnox@samba.org>
Signed-off-by: Andreas Schneider <asn@samba.org>
Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Diffstat (limited to 'source3/smbd')
| -rw-r--r-- | source3/smbd/dfree.c | 29 |
1 files changed, 13 insertions, 16 deletions
diff --git a/source3/smbd/dfree.c b/source3/smbd/dfree.c index bcefea5b274..12578407ce9 100644 --- a/source3/smbd/dfree.c +++ b/source3/smbd/dfree.c @@ -83,7 +83,7 @@ uint64_t sys_disk_free(connection_struct *conn, const char *path, DEBUG (3, ("disk_free: Running command '%s'\n", syscmd)); lines = file_lines_pload(syscmd, NULL); - if (lines) { + if (lines != NULL) { char *line = lines[0]; DEBUG (3, ("Read input from dfree, \"%s\"\n", line)); @@ -107,22 +107,18 @@ uint64_t sys_disk_free(connection_struct *conn, const char *path, *dsize = 2048; if (!*dfree) *dfree = 1024; - } else { - DEBUG (0, ("disk_free: file_lines_load() failed for " - "command '%s'. Error was : %s\n", - syscmd, strerror(errno) )); - if (sys_fsusage(path, dfree, dsize) != 0) { - DEBUG (0, ("disk_free: sys_fsusage() failed. Error was : %s\n", - strerror(errno) )); - return (uint64_t)-1; - } - } - } else { - if (sys_fsusage(path, dfree, dsize) != 0) { - DEBUG (0, ("disk_free: sys_fsusage() failed. Error was : %s\n", - strerror(errno) )); - return (uint64_t)-1; + + goto dfree_done; } + DEBUG (0, ("disk_free: file_lines_load() failed for " + "command '%s'. Error was : %s\n", + syscmd, strerror(errno) )); + } + + if (sys_fsusage(path, dfree, dsize) != 0) { + DEBUG (0, ("disk_free: sys_fsusage() failed. Error was : %s\n", + strerror(errno) )); + return (uint64_t)-1; } if (disk_quotas(path, &bsize_q, &dfree_q, &dsize_q)) { @@ -146,6 +142,7 @@ uint64_t sys_disk_free(connection_struct *conn, const char *path, *dfree = MAX(1,*dfree); } +dfree_done: disk_norm(bsize, dfree, dsize); if ((*bsize) < 1024) { |