[SCM] Samba Shared Repository - branch v3-2-test updated -
release-3-2-0pre2-3113-g6233913
Derrell Lipman
derrell at samba.org
Fri Oct 24 15:35:50 GMT 2008
The branch, v3-2-test has been updated
via 623391308f3fb26fdc2515baf41bb57ac1e5fc96 (commit)
from dcc40a6adec5d00c29a062164cbc68ff4a7779f8 (commit)
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test
- Log -----------------------------------------------------------------
commit 623391308f3fb26fdc2515baf41bb57ac1e5fc96
Author: Derrell Lipman <derrell.lipman at unwireduniverse.com>
Date: Fri Oct 24 11:35:10 2008 -0400
Error return is boolean false, not -1
- There were a few places in SMBC_getatr() that returned -1 instead of a
boolean. -1 was intended to mean error, but that's what False/false is for,
and the usages of this function assume that it returns a boolean false as
the error condition.
- per Jelmer's request, use false vs. False in new code, even if not making
changes globally.
Derrell
-----------------------------------------------------------------------
Summary of changes:
source/libsmb/libsmb_file.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source/libsmb/libsmb_file.c b/source/libsmb/libsmb_file.c
index 423450b..27b7e4f 100644
--- a/source/libsmb/libsmb_file.c
+++ b/source/libsmb/libsmb_file.c
@@ -518,7 +518,7 @@ SMBC_getatr(SMBCCTX * context,
errno = EINVAL;
TALLOC_FREE(frame);
- return -1;
+ return false;
}
/* path fixup for . and .. */
@@ -527,14 +527,14 @@ SMBC_getatr(SMBCCTX * context,
if (!fixedpath) {
errno = ENOMEM;
TALLOC_FREE(frame);
- return -1;
+ return false;
}
} else {
fixedpath = talloc_strdup(frame, path);
if (!fixedpath) {
errno = ENOMEM;
TALLOC_FREE(frame);
- return -1;
+ return false;
}
trim_string(fixedpath, NULL, "\\..");
trim_string(fixedpath, NULL, "\\.");
--
Samba Shared Repository
More information about the samba-cvs
mailing list