[SCM] Samba Shared Repository - branch master updated
Ralph Böhme
slow at samba.org
Wed Nov 30 23:29:03 UTC 2016
The branch, master has been updated
via c98bdf2 smbd/service_stream: connection processing flag is not really bool
via 28eb49c lib/registry/regf: better initialise nk_block
via f5154a8 ldb: Fix an unused variable warning
via b38c8da ldb: Fix typos
from 78a77d4 tdb: version 1.3.12
https://git.samba.org/?p=samba.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit c98bdf24941ad95250d79eeed26c57d600cedc7b
Author: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Date: Wed Nov 30 14:56:16 2016 +1300
smbd/service_stream: connection processing flag is not really bool
The warning is:
../source4/smbd/service_stream.c: In function ‘stream_io_handler’:
../source4/smbd/service_stream.c:94:18: warning: increment of a boolean expression [-Wbool-operation]
conn->processing++;
^~
../source4/smbd/service_stream.c:100:18: warning: decrement of a boolean expression [-Wbool-operation]
conn->processing--;
^~
while the code in question looks like:
conn->processing++;
if (flags & TEVENT_FD_WRITE) {
conn->ops->send_handler(conn, flags);
} else if (flags & TEVENT_FD_READ) {
conn->ops->recv_handler(conn, flags);
}
conn->processing--;
If this is never going to be nested, processing can be bool and the ++
and -- can be true/false assignments. But it seems possible that these
might be nested so it is better to go the other way.
Signed-off-by: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Reviewed-by: Ralph Boehme <slow at samba.org>
Autobuild-User(master): Ralph Böhme <slow at samba.org>
Autobuild-Date(master): Thu Dec 1 00:28:05 CET 2016 on sn-devel-144
commit 28eb49c7ede1c530f4c27fb4c136e88fd6ba7f93
Author: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Date: Wed Nov 30 14:19:39 2016 +1300
lib/registry/regf: better initialise nk_block
We were initialising a uint32_t[5] block with memset(..., 5) when we
surely meant memset(..., 5 * sizeof(uint32_t)) or some equivalent.
Thanks go to gcc-7 and -Wmemset-elt-size. The warning looks like this:
../source4/lib/registry/regf.c: In function ‘reg_create_regf_file’:
../source4/lib/registry/regf.c:2095:2: warning: ‘memset’ used with length equal to number of elements without multiplication by element size [-Wmemset-elt-size]
memset(nk.unk3, 0, 5);
^~~~~~
Signed-off-by: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Reviewed-by: Ralph Boehme <slow at samba.org>
commit f5154a86fab60b572f0b77017508a940c55d49e4
Author: Volker Lendecke <vl at samba.org>
Date: Mon Nov 21 21:41:51 2016 +0100
ldb: Fix an unused variable warning
Signed-off-by: Volker Lendecke <vl at samba.org>
Reviewed-by: Ralph Boehme <slow at samba.org>
commit b38c8da937e8129ee8096bd3059fcc1bb8f48633
Author: Volker Lendecke <vl at samba.org>
Date: Mon Nov 21 21:40:21 2016 +0100
ldb: Fix typos
Signed-off-by: Volker Lendecke <vl at samba.org>
Reviewed-by: Ralph Boehme <slow at samba.org>
-----------------------------------------------------------------------
Summary of changes:
lib/ldb/common/ldb_modules.c | 7 +++++--
source4/lib/registry/regf.c | 2 +-
source4/smbd/service_stream.h | 2 +-
3 files changed, 7 insertions(+), 4 deletions(-)
Changeset truncated at 500 lines:
diff --git a/lib/ldb/common/ldb_modules.c b/lib/ldb/common/ldb_modules.c
index 2105966..62f20af 100644
--- a/lib/ldb/common/ldb_modules.c
+++ b/lib/ldb/common/ldb_modules.c
@@ -901,7 +901,10 @@ static int ldb_modules_load_path(const char *path, const char *version)
} *loaded;
struct loaded *le;
int dlopen_flags;
+
+#ifdef RTLD_DEEPBIND
bool deepbind_enabled = (getenv("LDB_MODULES_DISABLE_DEEPBIND") == NULL);
+#endif
ret = stat(path, &st);
if (ret != 0) {
@@ -937,13 +940,13 @@ static int ldb_modules_load_path(const char *path, const char *version)
#ifdef RTLD_DEEPBIND
/*
* use deepbind if possible, to avoid issues with different
- * system library varients, for example ldb modules may be linked
+ * system library variants, for example ldb modules may be linked
* against Heimdal while the application may use MIT kerberos.
*
* See the dlopen manpage for details.
*
* One typical user is the bind_dlz module of Samba,
- * but symbol versioniong might be enough...
+ * but symbol versioning might be enough...
*
* We need a way to disable this in order to allow the
* ldb_*ldap modules to work with a preloaded socket wrapper.
diff --git a/source4/lib/registry/regf.c b/source4/lib/registry/regf.c
index 887b335..763b0ce 100644
--- a/source4/lib/registry/regf.c
+++ b/source4/lib/registry/regf.c
@@ -2092,7 +2092,7 @@ WERROR reg_create_regf_file(TALLOC_CTX *parent_ctx,
nk.unknown_offset = -1;
nk.num_values = 0;
nk.values_offset = -1;
- memset(nk.unk3, 0, 5);
+ memset(nk.unk3, 0, 5 * sizeof(uint32_t));
nk.clsname_offset = -1;
nk.clsname_length = 0;
nk.sk_offset = 0x80;
diff --git a/source4/smbd/service_stream.h b/source4/smbd/service_stream.h
index e098a69..a7d3def 100644
--- a/source4/smbd/service_stream.h
+++ b/source4/smbd/service_stream.h
@@ -60,7 +60,7 @@ struct stream_connection {
*/
struct auth_session_info *session_info;
- bool processing;
+ uint processing;
const char *terminate;
};
--
Samba Shared Repository
More information about the samba-cvs
mailing list