Skip to content

Commit

Permalink
s4:lib/messaging: use 'msg.lock' and 'msg.sock' for messaging related…
Browse files Browse the repository at this point in the history
… subdirs

In Samba 4.2, we used lock_path("msg") (with 0700) for the socket directory,
while we use lock_path("msg") (with 0755) for the lock file directory.

This generates a conflict that prevents samba, smbd, nmbd and winbindd
from starting after an upgrade.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=11515

Signed-off-by: Stefan Metzmacher <[email protected]>
Reviewed-by: Volker Lendecke <[email protected]>

Autobuild-User(master): Stefan Metzmacher <[email protected]>
Autobuild-Date(master): Thu Sep 17 09:04:59 CEST 2015 on sn-devel-104
  • Loading branch information
metze-samba committed Sep 17, 2015
1 parent 1aabd92 commit 1d2a1a6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions source4/lib/messaging/messaging.c
Original file line number Diff line number Diff line change
Expand Up @@ -323,7 +323,7 @@ struct imessaging_context *imessaging_init(TALLOC_CTX *mem_ctx,

/* create the messaging directory if needed */

msg->sock_dir = lpcfg_private_path(msg, lp_ctx, "sock");
msg->sock_dir = lpcfg_private_path(msg, lp_ctx, "msg.sock");
if (msg->sock_dir == NULL) {
goto fail;
}
Expand All @@ -332,7 +332,7 @@ struct imessaging_context *imessaging_init(TALLOC_CTX *mem_ctx,
goto fail;
}

msg->lock_dir = lpcfg_lock_path(msg, lp_ctx, "msg");
msg->lock_dir = lpcfg_lock_path(msg, lp_ctx, "msg.lock");
if (msg->lock_dir == NULL) {
goto fail;
}
Expand Down

0 comments on commit 1d2a1a6

Please sign in to comment.