[SCM] Samba Shared Repository - branch v4-20-test updated
Jule Anger
janger at samba.org
Fri Oct 4 12:56:01 UTC 2024
The branch, v4-20-test has been updated
via 676ac1793a1 s3: SIGHUP handlers use consistent log level 3
via 165149da928 shadow_copy2: Ignore VFS_OPEN_HOW_WITH_BACKUP_INTENT
from fa2041cef64 s4:lib/messaging: fix interaction between imessaging_reinit and irpc_destructor
https://git.samba.org/?p=samba.git;a=shortlog;h=v4-20-test
- Log -----------------------------------------------------------------
commit 676ac1793a1c13f45b9b76ee405511cf836cb000
Author: Jones Syue <jonessyue at qnap.com>
Date: Mon Sep 2 17:11:00 2024 +0800
s3: SIGHUP handlers use consistent log level 3
When turn-on 'log level = 3', sending SIGHUP to samba processes, for
example: smbd parent/children, smbd-notifyd, and smbd-cleanupd. Then
monitor log.smbd in order to parse sighup logs, it looks like the log level
is inconsistent among these processes: smbd parent/children use level 1,
and smbd-notifyd/smbd-cleanupd use level 3.
This patch raises sighup handler's log level from level 1 to level 3, which
is more consistent with smbd-notifyd by Commit 6e5bff80a0a0b ("s3:notifyd:
Handle sigup in notifyd to reparse smb.conf"), and smbd-cleanupd by Commit
57c1e115ecef4 ("smbd: reopen logs on SIGHUP for notifyd and cleanupd").
BUG: https://bugzilla.samba.org/show_bug.cgi?id=15706
Signed-off-by: Jones Syue <jonessyue at qnap.com>
Reviewed-by: Jeremy Allison <jra at samba.org>
Reviewed-by: Martin Schwenke <martin at meltin.net>
Autobuild-User(master): Martin Schwenke <martins at samba.org>
Autobuild-Date(master): Wed Sep 25 01:38:02 UTC 2024 on atb-devel-224
(cherry picked from commit 4f3dfb2029c667b6dcd43223fe154dca59143e95)
Autobuild-User(v4-20-test): Jule Anger <janger at samba.org>
Autobuild-Date(v4-20-test): Fri Oct 4 12:55:29 UTC 2024 on atb-devel-224
commit 165149da928e2fdc10f92e4fa52f3af787fe929d
Author: Christof Schmitt <cs at samba.org>
Date: Tue Oct 1 13:29:56 2024 -0700
shadow_copy2: Ignore VFS_OPEN_HOW_WITH_BACKUP_INTENT
d1846452e96 vfs: Add VFS_OPEN_HOW_WITH_BACKUP_INTENT introduced
VFS_OPEN_HOW_WITH_BACKUP_INTENT for files opened with
FILE_OPEN_FOR_BACKUP_INTENT. shadow_copy2 refuses the open on a file if
any flage in how.resolve is set. Change the check in shadow_copy2 to
allow opening of files with VFS_OPEN_HOW_WITH_BACKUP_INTENT.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=15730
Signed-off-by: Christof Schmitt <cs at samba.org>
Reviewed-by: Volker Lendecke <vl at samba.org>
Autobuild-User(master): Volker Lendecke <vl at samba.org>
Autobuild-Date(master): Wed Oct 2 08:06:38 UTC 2024 on atb-devel-224
(cherry picked from commit 30b0fa892ad66bfad92403186f97fd46496e62de)
-----------------------------------------------------------------------
Summary of changes:
source3/modules/vfs_shadow_copy2.c | 2 +-
source3/printing/queue_process.c | 2 +-
source3/smbd/server.c | 2 +-
source3/smbd/smb2_process.c | 2 +-
source3/winbindd/winbindd_dual.c | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source3/modules/vfs_shadow_copy2.c b/source3/modules/vfs_shadow_copy2.c
index c6e6ecd26c4..309be6e293e 100644
--- a/source3/modules/vfs_shadow_copy2.c
+++ b/source3/modules/vfs_shadow_copy2.c
@@ -1560,7 +1560,7 @@ static int shadow_copy2_openat(vfs_handle_struct *handle,
int ret;
bool ok;
- if (how.resolve != 0) {
+ if ((how.resolve & ~VFS_OPEN_HOW_WITH_BACKUP_INTENT) != 0) {
errno = ENOSYS;
return -1;
}
diff --git a/source3/printing/queue_process.c b/source3/printing/queue_process.c
index 6613e8f9f55..184fe16e61a 100644
--- a/source3/printing/queue_process.c
+++ b/source3/printing/queue_process.c
@@ -218,7 +218,7 @@ static void bq_sig_hup_handler(struct tevent_context *ev,
state = talloc_get_type_abort(pvt, struct bq_state);
change_to_root_user();
- DEBUG(1, ("Reloading pcap cache after SIGHUP\n"));
+ DBG_NOTICE("Reloading pcap cache after SIGHUP\n");
pcap_cache_reload(state->ev, state->msg,
reload_pcap_change_notify);
printing_subsystem_queue_tasks(state);
diff --git a/source3/smbd/server.c b/source3/smbd/server.c
index 42abfa948e6..e983efffa39 100644
--- a/source3/smbd/server.c
+++ b/source3/smbd/server.c
@@ -1417,7 +1417,7 @@ static void smbd_parent_sig_hup_handler(struct tevent_context *ev,
void *private_data)
{
change_to_root_user();
- DEBUG(1,("parent: Reloading services after SIGHUP\n"));
+ DBG_NOTICE("parent: Reloading services after SIGHUP\n");
reload_services(NULL, NULL, false);
}
diff --git a/source3/smbd/smb2_process.c b/source3/smbd/smb2_process.c
index d994a0ffd4b..aeb5b55586c 100644
--- a/source3/smbd/smb2_process.c
+++ b/source3/smbd/smb2_process.c
@@ -1644,7 +1644,7 @@ static void smbd_sig_hup_handler(struct tevent_context *ev,
struct smbd_server_connection);
change_to_root_user();
- DEBUG(1,("Reloading services after SIGHUP\n"));
+ DBG_NOTICE("Reloading services after SIGHUP\n");
reload_services(sconn, conn_snum_used, false);
}
diff --git a/source3/winbindd/winbindd_dual.c b/source3/winbindd/winbindd_dual.c
index e63b4052fd8..5f3239cec18 100644
--- a/source3/winbindd/winbindd_dual.c
+++ b/source3/winbindd/winbindd_dual.c
@@ -2062,7 +2062,7 @@ static void winbindd_sig_hup_handler(struct tevent_context *ev,
{
const char *file = (const char *)private_data;
- DEBUG(1,("Reloading services after SIGHUP\n"));
+ DBG_NOTICE("Reloading services after SIGHUP\n");
flush_caches_noinit();
winbindd_reload_services_file(file);
}
--
Samba Shared Repository
More information about the samba-cvs
mailing list