From 0a07b8ebfe797f062e50fbb901cd8040513af6d2 Mon Sep 17 00:00:00 2001 From: Jelmer Vernooij Date: Sun, 20 Jun 2010 13:16:30 +0200 Subject: testparm: Convert to Python. --- source4/scripting/bin/testparm | 204 ++++++++++++++++++++++++++++++++ source4/utils/testparm.c | 260 ----------------------------------------- source4/utils/wscript_build | 6 - 3 files changed, 204 insertions(+), 266 deletions(-) create mode 100755 source4/scripting/bin/testparm delete mode 100644 source4/utils/testparm.c diff --git a/source4/scripting/bin/testparm b/source4/scripting/bin/testparm new file mode 100755 index 0000000000..1162c3b687 --- /dev/null +++ b/source4/scripting/bin/testparm @@ -0,0 +1,204 @@ +#!/usr/bin/python +# vim: expandtab ft=python +# +# Unix SMB/CIFS implementation. +# Test validity of smb.conf +# Copyright (C) Karl Auer 1993, 1994-1998 +# +# Extensively modified by Andrew Tridgell, 1995 +# Converted to popt by Jelmer Vernooij (jelmer@nl.linux.org), 2002 +# Updated for Samba4 by Andrew Bartlett 2006 +# Converted to Python by Jelmer Vernooij 2010 +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . +# +# Testbed for loadparm.c/params.c +# +# This module simply loads a specified configuration file and +# if successful, dumps it's contents to stdout. Note that the +# operation is performed with DEBUGLEVEL at 3. +# +# Useful for a quick 'syntax check' of a configuration file. +# + +#/*********************************************** +# Here we do a set of 'hard coded' checks for bad +# configuration settings. +#************************************************/ +# + +import logging +import optparse +import os +import sys + +# Find right directory when running from source tree +sys.path.insert(0, "bin/python") + +import samba +from samba import getopt as options + +def do_global_checks(lp, logger): + ret = False + + lockdir = lp.get("lockdir") + + if not os.path.isdir(lockdir): + logger.error("lock directory %s does not exist", lockdir) + ret = True + + piddir = lp.get("pid directory") + + if os.path.isdir(piddir): + logger.error("pid directory %s does not exist", piddir) + ret = True + + winbind_separator = lp.get("winbind separator") + + if len(winbind_separator) != 1: + logger.error("the 'winbind separator' parameter must be a single character.") + ret = True + + if winbind_separator == '+': + logger.error("'winbind separator = +' might cause problems with group membership.") + + return ret + + +def allow_access(deny_list, allow_list, cname, caddr): + raise NotImplementedError(allow_access) + + +def do_share_checks(lp, cname, caddr, silent_mode, show_defaults, section_name, + parameter_name, logger): + ret = False + + for s in lp.services(): + if len(s) > 12: + logger.warning("You have some share names that are longer than 12 characters. These may not be accessible to some older clients. (Eg. Windows9x, WindowsMe, and not listed in smbclient in Samba 3.0.)") + break + + for s in lp.services(): + deny_list = lp.get("hosts deny", s) + allow_list = lp.get("hosts allow", s) + if deny_list: + for entry in deny_list: + if "*" in entry or "?" in entry: + logger.error("Invalid character (* or ?) in hosts deny list (%s) for service %s.", entry, s) + + if allow_list: + for entry in allow_list: + if "*" in entry or "?" in entry: + logger.error("Invalid character (* or ?) in hosts allow list (%s) for service %s.", entry, s) + + if cname is not None: + if not silent_mode: + print "Press enter to see a dump of your service definitions\n" + sys.stdin.readline() + if section_name is not None or parameter_name is not None: + if parameter_name is None: + lp.dump_service(sys.stdout, section_name, show_defaults) + else: + lp.dump_parameter(sys.stdout, section_name, parameter_name) + else: + lp.dump(sys.stdout, show_defaults) + return ret + + if cname is not None and caddr is not None: + # this is totally ugly, a real `quick' hack + for s in lp.services(): + if (allow_access(lp.get("hosts deny"), lp.get("hosts allow"), cname, caddr) and + allow_access(lp.get("hosts deny", s), lp.get("hosts allow", s), cname, caddr)): + logger.info("Allow connection from %s (%s) to %s", + cname, caddr, s) + else: + logger.info("Deny connection from %s (%s) to %s", + cname, caddr, s) + + return ret + + +if __name__ == '__main__': + section_name = None + parameter_name = None + silent_mode = False + show_defaults = False + + parser = optparse.OptionParser("testparm [OPTION...] [host-name] [host-ip]") + parser.add_option("--section-name", type="string", metavar="SECTION", + help="Limit testparm to a named section") + parser.add_option("--parameter-name", type="string", metavar="PARAMETER", + help="Limit testparm to a named parameter") + parser.add_option("--client-name", type="string", metavar="HOSTNAME", + help="Client DNS name for 'hosts allow' checking (should match reverse lookup)") + parser.add_option("--client-ip", type="string", metavar="IP", + help="Client IP address for 'hosts allow' checking") + parser.add_option("--suppress-prompt", action="store_true", + help="Suppress prompt for enter") + parser.add_option("--verbose", action="store_true", + help="Show default options too") + parser.add_option_group(options.VersionOptions(parser)) + # We need support for smb.conf macros before this will work again + parser.add_option("--server", type="string", + help="Set %%L macro to servername") + # These are harder to do with the new code structure + parser.add_option("--show-all-parameters", action="store_true", + help="Show the parameters, type, possible values") + + sambaopts = options.SambaOptions(parser) + parser.add_option_group(sambaopts) + + opts, args = parser.parse_args() + +# +# if (show_all_parameters) { +# show_parameter_list() +# exit(0) +# } + + if len(args) > 0: + cname = args[0] + else: + cname = None + if len(args) > 1: + caddr = args[1] + else: + caddr = None + + if cname is not None and caddr is None: + print "ERROR: For 'hosts allow' check you must specify both a DNS name and an IP address.\n" + sys.exit(1) + +# FIXME: We need support for smb.conf macros before this will work again +# +# if (new_local_machine) { +# set_local_machine_name(new_local_machine, True) +# } + + lp = sambaopts.get_loadparm() + + # We need this to force the output + samba.set_debug_level(2) + + logger = logging.getLogger("testparm") + + print "Loaded smb config files from %s\n" % lp.configfile + + logger.info("Loaded services file OK.") + + do_global_checks(lp, logger) + do_share_checks(lp, cname, caddr, silent_mode, show_defaults, + section_name, parameter_name, logger) + + sys.exit(0) diff --git a/source4/utils/testparm.c b/source4/utils/testparm.c deleted file mode 100644 index c8e07973bc..0000000000 --- a/source4/utils/testparm.c +++ /dev/null @@ -1,260 +0,0 @@ -/* - Unix SMB/CIFS implementation. - Test validity of smb.conf - Copyright (C) Karl Auer 1993, 1994-1998 - - Extensively modified by Andrew Tridgell, 1995 - Converted to popt by Jelmer Vernooij (jelmer@nl.linux.org), 2002 - Updated for Samba4 by Andrew Bartlett 2006 - - This program is free software; you can redistribute it and/or modify - it under the terms of the GNU General Public License as published by - the Free Software Foundation; either version 3 of the License, or - (at your option) any later version. - - This program is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - GNU General Public License for more details. - - You should have received a copy of the GNU General Public License - along with this program. If not, see . -*/ - -/* - * Testbed for loadparm.c/params.c - * - * This module simply loads a specified configuration file and - * if successful, dumps it's contents to stdout. Note that the - * operation is performed with DEBUGLEVEL at 3. - * - * Useful for a quick 'syntax check' of a configuration file. - * - */ - -#include "includes.h" -#include "system/filesys.h" -#include "lib/cmdline/popt_common.h" -#include "lib/socket/socket.h" -#include "param/param.h" -#include "param/loadparm.h" - - -/*********************************************** - Here we do a set of 'hard coded' checks for bad - configuration settings. -************************************************/ - -static int do_global_checks(struct loadparm_context *lp_ctx) -{ - int ret = 0; - - if (!directory_exist(lp_lockdir(lp_ctx))) { - fprintf(stderr, "ERROR: lock directory %s does not exist\n", - lp_lockdir(lp_ctx)); - ret = 1; - } - - if (!directory_exist(lp_piddir(lp_ctx))) { - fprintf(stderr, "ERROR: pid directory %s does not exist\n", - lp_piddir(lp_ctx)); - ret = 1; - } - - if (strlen(lp_winbind_separator(lp_ctx)) != 1) { - fprintf(stderr,"ERROR: the 'winbind separator' parameter must be a single character.\n"); - ret = 1; - } - - if (*lp_winbind_separator(lp_ctx) == '+') { - fprintf(stderr,"'winbind separator = +' might cause problems with group membership.\n"); - } - - return ret; -} - - -static int do_share_checks(struct loadparm_context *lp_ctx, const char *cname, const char *caddr, bool silent_mode, - bool show_defaults, const char *section_name, const char *parameter_name) -{ - int ret = 0; - int s; - - for (s=0;s 12) { - fprintf(stderr, "WARNING: You have some share names that are longer than 12 characters.\n" ); - fprintf(stderr, "These may not be accessible to some older clients.\n" ); - fprintf(stderr, "(Eg. Windows9x, WindowsMe, and not listed in smbclient in Samba 3.0.)\n" ); - break; - } - } - - for (s=0;s