diff options
| author | Andrew Tridgell <tridge@samba.org> | 2010-11-11 10:35:38 +1100 |
|---|---|---|
| committer | Andrew Tridgell <tridge@samba.org> | 2010-11-11 02:57:04 +0000 |
| commit | ed8ea4ed18f08bfc93190513f2a2e8e058655daf (patch) | |
| tree | ccebab2f84ec3639a3681126893f7fcd6c337fbc /source4/param | |
| parent | 0afb2995a2177afa2eb7b8f99887a39cdaf23a15 (diff) | |
| download | samba-ed8ea4ed18f08bfc93190513f2a2e8e058655daf.tar.gz samba-ed8ea4ed18f08bfc93190513f2a2e8e058655daf.tar.bz2 samba-ed8ea4ed18f08bfc93190513f2a2e8e058655daf.zip | |
s4-server: move the creation of the IPC$ share into ntvfs
the IPC$ share is only used by the ntvfs backends, and doesn't need to
be created on every load of smb.conf. This fixes a problem with
testparm showing the ipc$ share when it isn't defined in smb.conf.
This also removes the admin$ share, which really shouldn't be on by
default. The admin$ share is used for remote software installation,
and normally exposes the c:\windows directory on a windows
server. That makes no sense on Samba. If for some reason a admin$
share is needed, then the admin can create one as usual. Exposing /tmp
via admin$ by default seems like a bad idea.
Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'source4/param')
| -rw-r--r-- | source4/param/loadparm.c | 36 |
1 files changed, 0 insertions, 36 deletions
diff --git a/source4/param/loadparm.c b/source4/param/loadparm.c index 69dead77a3..c85126ffae 100644 --- a/source4/param/loadparm.c +++ b/source4/param/loadparm.c @@ -1143,39 +1143,6 @@ bool lpcfg_add_home(struct loadparm_context *lp_ctx, } /** - * Add the IPC service. - */ - -static bool lpcfg_add_hidden(struct loadparm_context *lp_ctx, const char *name, - const char *fstype) -{ - struct loadparm_service *service = lpcfg_add_service(lp_ctx, lp_ctx->sDefault, name); - - if (service == NULL) - return false; - - string_set(service, &service->szPath, tmpdir()); - - service->comment = talloc_asprintf(service, "%s Service (%s)", - fstype, lp_ctx->globals->szServerString); - string_set(service, &service->fstype, fstype); - service->iMaxConnections = -1; - service->bAvailable = true; - service->bRead_only = true; - service->bPrint_ok = false; - service->bBrowseable = false; - - if (strcasecmp(fstype, "IPC") == 0) { - lpcfg_do_service_parameter(lp_ctx, service, "ntvfs handler", - "default"); - } - - DEBUG(3, ("adding hidden service %s\n", name)); - - return true; -} - -/** * Add a new printer service, with defaults coming from service iFrom. */ @@ -2501,9 +2468,6 @@ static bool lpcfg_update(struct loadparm_context *lp_ctx) { lpcfg_add_auto_services(lp_ctx, lpcfg_auto_services(lp_ctx)); - lpcfg_add_hidden(lp_ctx, "IPC$", "IPC"); - lpcfg_add_hidden(lp_ctx, "ADMIN$", "DISK"); - if (!lp_ctx->globals->szWINSservers && lp_ctx->globals->bWINSsupport) { lpcfg_do_global_parameter(lp_ctx, "wins server", "127.0.0.1"); } |