From fb924d02ec05f7106beeeb409becbc39e00ddac9 Mon Sep 17 00:00:00 2001 From: Christian Ambach Date: Mon, 11 Mar 2013 22:23:27 +0100 Subject: s3: remove some dead code (for setdir command) set dir seems to have been a special SMB command used by Pathworks clients the supporting code for it was already removed in 2007, so just remove all remnants related to it (smb.conf parameter, documentation, ...) Reviewed-by: Jeremy Allison Autobuild-User(master): Jeremy Allison Autobuild-Date(master): Tue Mar 12 01:03:37 CET 2013 on sn-devel-104 --- docs-xml/smbdotconf/misc/setdirectory.xml | 19 ------------------- docs-xml/using_samba/ch08.xml | 30 +++++------------------------- examples/scripts/shares/python/smbparm.py | 1 - lib/param/param_functions.c | 1 - lib/param/param_table.c | 9 --------- source3/include/proto.h | 1 - source3/include/smb_macros.h | 1 - source3/include/smbprofile.h | 5 ----- source3/param/loadparm.c | 1 - source3/profile/profile.c | 1 - source3/utils/status_profile.c | 4 ---- 11 files changed, 5 insertions(+), 68 deletions(-) delete mode 100644 docs-xml/smbdotconf/misc/setdirectory.xml diff --git a/docs-xml/smbdotconf/misc/setdirectory.xml b/docs-xml/smbdotconf/misc/setdirectory.xml deleted file mode 100644 index 5e7b379a04..0000000000 --- a/docs-xml/smbdotconf/misc/setdirectory.xml +++ /dev/null @@ -1,19 +0,0 @@ - - - - If set directory = no, then users of the - service may not use the setdir command to change directory. - - - - The setdir command is only implemented - in the Digital Pathworks client. See the Pathworks documentation - for details. - - - -no - diff --git a/docs-xml/using_samba/ch08.xml b/docs-xml/using_samba/ch08.xml index 91e216a25c..7465b8a67b 100644 --- a/docs-xml/using_samba/ch08.xml +++ b/docs-xml/using_samba/ch08.xml @@ -1795,27 +1795,7 @@ panic action - - - - -set directory option - -set directory - - -This boolean share-level option allows Digital Pathworks clients, option forDigital Pathworks clients to use the setdir command to change directories on the server. If you are not using the Digital Pathworks client, you should not need to alter this option. The default value for this option is no. You can override it per share as follows: - - -[data] - set directory = yes - - - - - - - + smbrun option smbrun @@ -1832,7 +1812,7 @@ smbrun - + status option status @@ -1850,7 +1830,7 @@ status - + strict sync option strict sync @@ -1867,7 +1847,7 @@ strict sync - + sync always option sync always @@ -1884,7 +1864,7 @@ sync always - + strip dot option strip dot diff --git a/examples/scripts/shares/python/smbparm.py b/examples/scripts/shares/python/smbparm.py index f0bc1ecb89..287b3572c0 100644 --- a/examples/scripts/shares/python/smbparm.py +++ b/examples/scripts/shares/python/smbparm.py @@ -68,7 +68,6 @@ parm_table = { "USESPNEGO" : ("use spnego", SambaParmBool, P_GLOBAL, "Yes"), "FAKEOPLOCKS" : ("fake oplocks", SambaParmBool, P_LOCAL, "No"), "FORCECREATEMODE" : ("force create mode", SambaParmString, P_LOCAL, "00"), - "SETDIRECTORY" : ("set directory", SambaParmBoolRev, P_LOCAL, "No"), "SMBPORTS" : ("smb ports", SambaParmString, P_GLOBAL, "445 139"), "LOCKDIR" : ("lock directory", SambaParmString, P_GLOBAL, "/var/lib/samba"), "BROWSEABLE" : ("browseable", SambaParmBool, P_LOCAL, "Yes"), diff --git a/lib/param/param_functions.c b/lib/param/param_functions.c index 4cf3ad7698..7cb3a69112 100644 --- a/lib/param/param_functions.c +++ b/lib/param/param_functions.c @@ -92,7 +92,6 @@ FN_LOCAL_BOOL(hide_special_files, bHideSpecialFiles) FN_LOCAL_BOOL(hideunreadable, bHideUnReadable) FN_LOCAL_BOOL(hideunwriteable_files, bHideUnWriteableFiles) FN_LOCAL_BOOL(access_based_share_enum, bAccessBasedShareEnum) -FN_LOCAL_BOOL(no_set_dir, bNo_set_dir) FN_LOCAL_BOOL(guest_ok, bGuest_ok) FN_LOCAL_BOOL(guest_only, bGuest_only) FN_LOCAL_BOOL(administrative_share, bAdministrative_share) diff --git a/lib/param/param_table.c b/lib/param/param_table.c index 620f5f679b..c65a73824f 100644 --- a/lib/param/param_table.c +++ b/lib/param/param_table.c @@ -3725,15 +3725,6 @@ static struct parm_struct parm_table[] = { .enum_list = NULL, .flags = FLAG_ADVANCED | FLAG_SHARE, }, - { - .label = "set directory", - .type = P_BOOLREV, - .p_class = P_LOCAL, - .offset = LOCAL_VAR(bNo_set_dir), - .special = NULL, - .enum_list = NULL, - .flags = FLAG_ADVANCED | FLAG_SHARE, - }, { .label = "allow insecure wide links", .type = P_BOOL, diff --git a/source3/include/proto.h b/source3/include/proto.h index c36125e53e..00a9a69d73 100644 --- a/source3/include/proto.h +++ b/source3/include/proto.h @@ -1278,7 +1278,6 @@ bool lp_hideunwriteable_files(int ); bool lp_browseable(int ); bool lp_access_based_share_enum(int ); bool lp_readonly(int ); -bool lp_no_set_dir(int ); bool lp_guest_ok(int ); bool lp_guest_only(int ); bool lp_administrative_share(int ); diff --git a/source3/include/smb_macros.h b/source3/include/smb_macros.h index 8b9d94181f..ff3aced755 100644 --- a/source3/include/smb_macros.h +++ b/source3/include/smb_macros.h @@ -72,7 +72,6 @@ #define VALID_SNUM(snum) (lp_snum_ok(snum)) #define GUEST_OK(snum) (VALID_SNUM(snum) && lp_guest_ok(snum)) #define GUEST_ONLY(snum) (VALID_SNUM(snum) && lp_guest_only(snum)) -#define CAN_SETDIR(snum) (!lp_no_set_dir(snum)) #define CAN_PRINT(conn) ((conn) && lp_print_ok(SNUM(conn))) #define MAP_HIDDEN(conn) ((conn) && lp_map_hidden(SNUM(conn))) #define MAP_SYSTEM(conn) ((conn) && lp_map_system(SNUM(conn))) diff --git a/source3/include/smbprofile.h b/source3/include/smbprofile.h index 6e136a08fc..9bcee42f8e 100644 --- a/source3/include/smbprofile.h +++ b/source3/include/smbprofile.h @@ -563,11 +563,6 @@ enum profile_stats_values #define SMBinvalid_count __profile_stats_value(PR_VALUE_SMBINVALID, count) #define SMBinvalid_time __profile_stats_value(PR_VALUE_SMBINVALID, time) -/* Pathworks setdir command */ - PR_VALUE_PATHWORKS_SETDIR, -#define pathworks_setdir_count __profile_stats_value(PR_VALUE_PATHWORKS_SETDIR, count) -#define pathworks_setdir_time __profile_stats_value(PR_VALUE_PATHWORKS_SETDIR, time) - /* These are the TRANS2 sub commands */ PR_VALUE_TRANS2_OPEN, #define Trans2_open_count __profile_stats_value(PR_VALUE_TRANS2_OPEN, count) diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c index 75b63c5231..d1c2a6383e 100644 --- a/source3/param/loadparm.c +++ b/source3/param/loadparm.c @@ -214,7 +214,6 @@ static struct loadparm_service sDefault = .bAccessBasedShareEnum = false, .bAvailable = true, .bRead_only = true, - .bNo_set_dir = true, .bGuest_only = false, .bAdministrative_share = false, .bGuest_ok = false, diff --git a/source3/profile/profile.c b/source3/profile/profile.c index 0287860ff2..5f9433f999 100644 --- a/source3/profile/profile.c +++ b/source3/profile/profile.c @@ -329,7 +329,6 @@ bool profile_setup(struct messaging_context *msg_ctx, bool rdonly) "SMBsendend", /* PR_VALUE_SMBSENDEND */ "SMBsendtxt", /* PR_VALUE_SMBSENDTXT */ "SMBinvalid", /* PR_VALUE_SMBINVALID */ - "pathworks_setdir", /* PR_VALUE_PATHWORKS_SETDIR */ "Trans2_open", /* PR_VALUE_TRANS2_OPEN */ "Trans2_findfirst", /* PR_VALUE_TRANS2_FINDFIRST */ "Trans2_findnext", /* PR_VALUE_TRANS2_FINDNEXT */ diff --git a/source3/utils/status_profile.c b/source3/utils/status_profile.c index e01b1652ab..0f722196a5 100644 --- a/source3/utils/status_profile.c +++ b/source3/utils/status_profile.c @@ -298,10 +298,6 @@ bool status_profile_dump(bool verbose) d_printf("invalid_count: %u\n", profile_p->SMBinvalid_count); d_printf("invalid_time: %u\n", profile_p->SMBinvalid_time); - profile_separator("Pathworks Calls"); - d_printf("setdir_count: %u\n", profile_p->pathworks_setdir_count); - d_printf("setdir_time: %u\n", profile_p->pathworks_setdir_time); - profile_separator("Trans2 Calls"); d_printf("open_count: %u\n", profile_p->Trans2_open_count); d_printf("open_time: %u\n", profile_p->Trans2_open_time); -- cgit