From 73a2738eba0625fa9befcaeed690e035b12ab91e Mon Sep 17 00:00:00 2001 From: SARL ENR 68 Date: Fri, 28 Aug 2015 19:58:30 +0200 Subject: Adaptation to python3 using 2to3-3.4 (mga#16524) - by papoteur: https://bugs.mageia.org/attachment.cgi?id=6898 --- src/msec/compile.py | 2 +- src/msec/config.py | 26 ++++++++++---------- src/msec/help_draksec.py | 18 +++++++------- src/msec/libmsec.py | 58 ++++++++++++++++++++++----------------------- src/msec/man.py | 10 ++++---- src/msec/msec.py | 16 ++++++------- src/msec/msecgui.py | 44 +++++++++++++++++----------------- src/msec/msecperms.py | 18 +++++++------- src/msec/plugins/audit.py | 2 +- src/msec/plugins/log.py | 2 +- src/msec/plugins/msec.py | 6 ++--- src/msec/plugins/network.py | 2 +- src/msec/plugins/pam.py | 2 +- src/msec/plugins/sectool.py | 2 +- src/msec/plugins/sudo.py | 2 +- src/msec/tools.py | 6 ++--- 16 files changed, 108 insertions(+), 108 deletions(-) (limited to 'src') diff --git a/src/msec/compile.py b/src/msec/compile.py index b85e3f9..150a180 100755 --- a/src/msec/compile.py +++ b/src/msec/compile.py @@ -15,7 +15,7 @@ for f in sys.argv[2:]: try: py_compile.compile(f, f+"o", sys.argv[1] + f) except: - print("Cannot compile", f) + print(("Cannot compile", f)) pass # compile.py ends here diff --git a/src/msec/config.py b/src/msec/config.py index 9f41998..8caacfc 100755 --- a/src/msec/config.py +++ b/src/msec/config.py @@ -38,7 +38,7 @@ SECURITYLOG = '/var/log/msec.log' # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str @@ -267,7 +267,7 @@ class MsecConfig: def list_options(self): """Sorts and returns configuration parameters""" - sortedparams = self.options.keys() + sortedparams = list(self.options.keys()) if sortedparams: sortedparams.sort() return sortedparams @@ -280,10 +280,10 @@ class MsecConfig: try: fd = open(self.config, "w") except: - self.log.error(_("Unable to save %s: %s") % (self.config, Narg(sys.exc_value))) + self.log.error(_("Unable to save %s: %s") % (self.config, Narg(sys.exc_info()[1]))) return False for comment in self.comments: - print >>fd, comment + print(comment, file=fd) # sorting keys for option in self.list_options(): value = self.options[option] @@ -310,7 +310,7 @@ class MsecConfig: pass else: value = '"%s"' % value - print >>fd, "%s=%s" % (option, value) + print("%s=%s" % (option, value), file=fd) return True # }}} @@ -373,7 +373,7 @@ class ExceptionConfig: def set(self, pos, value): """Sets a configuration option""" if pos > 0: - print "Pos: %d" % pos + print("Pos: %d" % pos) self.options[pos] = value else: self.options.append(value) @@ -393,17 +393,17 @@ class ExceptionConfig: try: fd = open(self.config, "w") except: - self.log.error(_("Unable to save %s: %s") % (self.config, Narg(sys.exc_value))) + self.log.error(_("Unable to save %s: %s") % (self.config, Narg(sys.exc_info()[1]))) return False for comment in self.comments: - print >>fd, comment + print(comment, file=fd) # sorting keys for option,value in self.options: # TODO: integrate with remove() if value == None or value == OPTION_DISABLED: self.log.debug("Skipping %s" % option) else: - print >>fd, "%s %s" % (option, value) + print("%s %s" % (option, value), file=fd) return True # }}} @@ -443,7 +443,7 @@ class PermConfig(MsecConfig): try: fd = open(self.config) except: - self.log.error(_("Unable to load configuration file %s: %s") % (self.config, Narg(sys.exc_value))) + self.log.error(_("Unable to load configuration file %s: %s") % (self.config, Narg(sys.exc_info()[1]))) return False for line in fd.readlines(): line = line.strip() @@ -488,10 +488,10 @@ class PermConfig(MsecConfig): try: fd = open(self.config, "w") except: - self.log.error(_("Unable to save %s: %s") % (self.config, Narg(sys.exc_value))) + self.log.error(_("Unable to save %s: %s") % (self.config, Narg(sys.exc_info()[1]))) return False for comment in self.comments: - print >>fd, comment + print(comment, file=fd) # sorting keys for file in self.options_order: value = self.options[file] @@ -509,6 +509,6 @@ class PermConfig(MsecConfig): force = "\tforce" else: force = "" - print >>fd, "%s\t%s.%s\t%s%s\t%s" % (file, user, group, perm, force, acl) + print("%s\t%s.%s\t%s%s\t%s" % (file, user, group, perm, force, acl), file=fd) return True # }}} diff --git a/src/msec/help_draksec.py b/src/msec/help_draksec.py index 3eb4a6d..f4536a3 100755 --- a/src/msec/help_draksec.py +++ b/src/msec/help_draksec.py @@ -81,20 +81,20 @@ help_wiki = open(help_wiki_file, "w") log = Log(log_syslog=False, log_file=False) msec = MSEC(log, plugins='plugins') -print >>help_perl, header_perl -print >>help_python, header_python -print >>help_wiki, header_wiki +print(header_perl, file=help_perl) +print(header_python, file=help_python) +print(header_wiki, file=help_wiki) for variable in config.SETTINGS: callback, params = config.SETTINGS[variable] func = msec.get_action(callback) if func: - print >>help_perl, function_str_perl % (variable, func.__doc__.strip()) - print >>help_python, function_str_python % (variable, func.__doc__.strip()) - print >>help_wiki, function_str_wiki % (variable, func.__doc__.strip()) + print(function_str_perl % (variable, func.__doc__.strip()), file=help_perl) + print(function_str_python % (variable, func.__doc__.strip()), file=help_python) + print(function_str_wiki % (variable, func.__doc__.strip()), file=help_wiki) -print >>help_perl, footer_perl -print >>help_python, footer_python -print >>help_wiki, footer_wiki +print(footer_perl, file=help_perl) +print(footer_python, file=help_python) +print(footer_wiki, file=help_wiki) # draksec_help.py ends here diff --git a/src/msec/libmsec.py b/src/msec/libmsec.py index 593c46a..79a1ff7 100755 --- a/src/msec/libmsec.py +++ b/src/msec/libmsec.py @@ -40,7 +40,7 @@ import gettext import pwd import re import string -import commands +import subprocess import time import stat import traceback @@ -57,7 +57,7 @@ from config import Narg # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str @@ -81,7 +81,7 @@ def move(old, new): try: os.rename(old, new) except: - error('rename %s %s: %s' % (old, new, str(sys.exc_value))) + error('rename %s %s: %s' % (old, new, str(sys.exc_info()[1]))) def substitute_re_result(res, s): for idx in range(0, (res.lastindex or 0) + 1): @@ -123,7 +123,7 @@ class Log: self.syslog_h.setFormatter(formatter) self.logger.addHandler(self.syslog_h) except: - print >>sys.stderr, "Logging to syslog not available: %s" % (sys.exc_value[1]) + print("Logging to syslog not available: %s" % (sys.exc_value[1]), file=sys.stderr) interactive = True # log to file @@ -134,7 +134,7 @@ class Log: self.file_h.setFormatter(formatter) self.logger.addHandler(self.file_h) except: - print >>sys.stderr, "Logging to '%s' not available: %s" % (self.log_path, sys.exc_value[1]) + print("Logging to '%s' not available: %s" % (self.log_path, sys.exc_value[1]), file=sys.stderr) interactive = True # interactive logging @@ -247,7 +247,7 @@ class ConfigFiles: def write_files(self, commit=True): """Writes all files back to disk""" - for f in self.files.values(): + for f in list(self.files.values()): self.log.debug("Attempting to write %s" % f.path) if commit: f.write() @@ -264,7 +264,7 @@ class ConfigFiles: s = substitute_re_result(res, a[1]) if commit: self.log.info(_('%s modified so launched command: %s') % (Narg(f), Narg(s))) - cmd = commands.getstatusoutput(s) + cmd = subprocess.getstatusoutput(s) cmd = [0, ''] if cmd[0] == 0: if cmd[1]: @@ -368,7 +368,7 @@ class ConfigFile: return link def write(self): - if not (type(self.path) == unicode): + if not (type(self.path) == str): self.pathN = Narg(self.path) else: self.pathN = self.path @@ -378,20 +378,20 @@ class ConfigFile: try: os.unlink(self.path) except: - error('unlink %s: %s' % (self.pathN, str(sys.exc_value))) + error('unlink %s: %s' % (self.pathN, str(sys.exc_info()[1]))) self.log.info(_('deleted %s') % (self.pathN,)) elif self.is_touched: if os.path.exists(self.path): try: os.utime(self.path, None) except: - self.log.error('utime %s: %s' % (self.pathN, str(sys.exc_value))) + self.log.error('utime %s: %s' % (self.pathN, str(sys.exc_info()[1]))) elif self.suffix and os.path.exists(self.path + self.suffix): move(self.path + self.suffix, self.path) try: os.utime(self.path, None) except: - self.log.error('utime %s: %s' % (self.pathN, str(sys.exc_value))) + self.log.error('utime %s: %s' % (self.pathN, str(sys.exc_info()[1]))) else: self.lines = [] self.is_modified = 1 @@ -399,7 +399,7 @@ class ConfigFile: file.close() self.log.info(_('touched file %s') % (self.pathN,)) elif self.sym_link: - if not (type(self.sym_link) == unicode): + if not (type(self.sym_link) == str): self.sym_linkN = Narg(self.sym_link) else: self.sym_linkN = self.sym_link @@ -414,16 +414,16 @@ class ConfigFile: try: os.unlink(self.path) except: - self.log.error('unlink %s: %s' % (self.pathN, str(sys.exc_value))) + self.log.error('unlink %s: %s' % (self.pathN, str(sys.exc_info()[1]))) self.log.info(_('deleted %s') % (self.pathN,)) if not done: try: os.symlink(self.sym_link, self.path) except: - self.log.error('symlink %s %s: %s' % (self.sym_linkN, self.path, str(sys.exc_value))) + self.log.error('symlink %s %s: %s' % (self.sym_linkN, self.path, str(sys.exc_info()[1]))) self.log.info(_('made symbolic link from %s to %s') % (self.sym_linkN, self.path)) elif self.is_moved: - if not (type(self.suffix) == unicode): + if not (type(self.suffix) == str): self.suffixN = Narg(self.suffix) else: self.suffixN = self.suffix @@ -696,7 +696,7 @@ class MSEC: self.plugins[plugin_name] = plugin self.log.debug("Loaded plugin '%s'" % plugin_f) except: - self.log.error(_("Error loading plugin '%s' from %s: %s") % (plugin_f, file, Narg(sys.exc_value))) + self.log.error(_("Error loading plugin '%s' from %s: %s") % (plugin_f, file, Narg(sys.exc_info()[1]))) def reset(self): """Resets the configuration""" @@ -885,7 +885,7 @@ class PERMS: try: os.chown(file, newuser, -1) except: - self.log.error(_("Error changing user on %s: %s") % (file, Narg(sys.exc_value))) + self.log.error(_("Error changing user on %s: %s") % (file, Narg(sys.exc_info()[1]))) else: self.log.warn(_("Wrong owner of %s: should be %s") % (file, self.get_user_name(newuser))) if newgroup != None: @@ -894,7 +894,7 @@ class PERMS: try: os.chown(file, -1, newgroup) except: - self.log.error(_("Error changing group on %s: %s") % (file, Narg(sys.exc_value))) + self.log.error(_("Error changing group on %s: %s") % (file, Narg(sys.exc_info()[1]))) else: self.log.warn(_("Wrong group of %s: should be %s") % (file, self.get_group_name(newgroup))) # permissions should be last, as chown resets them @@ -905,7 +905,7 @@ class PERMS: try: os.chmod(file, newperm) except: - self.log.error(_("Error changing permissions on %s: %s") % (file, Narg(sys.exc_value))) + self.log.error(_("Error changing permissions on %s: %s") % (file, Narg(sys.exc_info()[1]))) else: self.log.warn(_("Wrong permissions of %s: should be %o") % (file, newperm)) @@ -919,7 +919,7 @@ class PERMS: users = newacl.split(",") for acluser in users : if acluser.split(":")[0] == "": # clean root from list - print acluser + print(acluser) continue # make the acl rule stick ret = os.system('setfacl -m u:%s %s' % (acluser, file)) @@ -927,7 +927,7 @@ class PERMS: # problem setting setfacl self.log.error(_("Unable to add filesystem-specific ACL %s to %s") % (acluser, file)) except: - self.log.error(_("Error changing acl on %s: %s") % (file, Narg(str(sys.exc_value)))) + self.log.error(_("Error changing acl on %s: %s") % (file, Narg(str(sys.exc_info()[1])))) else: self.log.warn(_("Wrong acl of %s") % (file)) @@ -978,12 +978,12 @@ class PERMS: mode = stat.S_IMODE(full[stat.ST_MODE]) if perm != -1 and stat.S_ISDIR(full[stat.ST_MODE]): - if curperm & 0400: - curperm = curperm | 0100 - if curperm & 0040: - curperm = curperm | 0010 - if curperm & 0004: - curperm = curperm | 0001 + if curperm & 0o400: + curperm = curperm | 0o100 + if curperm & 0o040: + curperm = curperm | 0o010 + if curperm & 0o004: + curperm = curperm | 0o001 curuser = full[stat.ST_UID] curgroup = full[stat.ST_GID] @@ -1022,7 +1022,7 @@ class PERMS: paths_to_check.extend(glob.glob(f)) paths_to_check = set(paths_to_check) # remove unneeded entries from self.files - for f in self.files.keys(): + for f in list(self.files.keys()): if f not in paths_to_check: del self.files[f] return self.files @@ -1030,5 +1030,5 @@ class PERMS: if __name__ == "__main__": # this should never ever be run directly - print >>sys.stderr, """This file should not be run directly.""" + print("""This file should not be run directly.""", file=sys.stderr) diff --git a/src/msec/man.py b/src/msec/man.py index cc1a2f0..a937b07 100755 --- a/src/msec/man.py +++ b/src/msec/man.py @@ -288,12 +288,12 @@ msec = MSEC(log) #print >>sys.stderr, dir(msec.create_server_link) -print header +print(header) # sorting settings according to plugin callbacks = [] settings_rev = {} -for entry in config.SETTINGS.keys(): +for entry in list(config.SETTINGS.keys()): callback, params = config.SETTINGS[entry] callbacks.append(callback) settings_rev[callback] = (entry, params) @@ -304,11 +304,11 @@ for callback in callbacks: variable, params = settings_rev[callback] func = msec.get_action(callback) if func: - print function_str % (callback, func.__doc__.strip(), variable, ", ".join(params)) + print(function_str % (callback, func.__doc__.strip(), variable, ", ".join(params))) if variable in config.CHECKS_WITH_EXCEPTIONS: # this check supports exceptions - print """(This check supports exceptions via %s variable defined in \\fB/etc/security/msec/exceptions\\fP file)""" % variable + print("""(This check supports exceptions via %s variable defined in \\fB/etc/security/msec/exceptions\\fP file)""" % variable) -print footer +print(footer) # man.py ends here diff --git a/src/msec/msec.py b/src/msec/msec.py index bbcd4b6..f98a793 100755 --- a/src/msec/msec.py +++ b/src/msec/msec.py @@ -28,14 +28,14 @@ import logging # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str # {{{ usage def usage(): """Prints help message""" - print """Msec: Mageia Security Center (%s). + print("""Msec: Mageia Security Center (%s). When run without parameters, msec will read the configuration from /etc/security/msec/msec.conf, and enforce the specified security settings. @@ -54,7 +54,7 @@ Arguments to msec: -r, --root path to use as root -q, --quiet run quietly -s, --save save current configuration as a new security level -""" % version +""" % version) # }}} if __name__ == "__main__": @@ -84,10 +84,10 @@ if __name__ == "__main__": levelconf = config.load_defaults(log, level) params = levelconf.list_options() if not params: - print >>sys.stderr, _("Invalid security level '%s'.") % level + print(_("Invalid security level '%s'.") % level, file=sys.stderr) sys.exit(1) for item in params: - print "%s=%s" % (item, levelconf.get(item) ) + print("%s=%s" % (item, levelconf.get(item) )) sys.exit(0) # force new level elif o[0] == '-f' or o[0] == '--force': @@ -111,9 +111,9 @@ if __name__ == "__main__": # verifying use id if os.geteuid() != 0: - print >>sys.stderr, _("Msec: Mageia Security Center (%s)\n") % version - print >>sys.stderr, _("Error: This application must be executed by root!") - print >>sys.stderr, _("Run with --help to get help.") + print(_("Msec: Mageia Security Center (%s)\n") % version, file=sys.stderr) + print(_("Error: This application must be executed by root!"), file=sys.stderr) + print(_("Run with --help to get help."), file=sys.stderr) sys.exit(1) # configuring logging diff --git a/src/msec/msecgui.py b/src/msec/msecgui.py index d468b15..765681d 100755 --- a/src/msec/msecgui.py +++ b/src/msec/msecgui.py @@ -9,7 +9,7 @@ import string import getopt import signal import traceback -import Queue +import queue from textwrap import wrap from threading import Thread import time @@ -26,9 +26,9 @@ try: import gi from gi.repository import GObject from gi.repository import Pango -except Warning, e: - print "ERROR: %s" % e - print "Exiting.." +except Warning as e: + print("ERROR: %s" % e) + print("Exiting..") sys.exit(1) warnings.resetwarnings() @@ -51,7 +51,7 @@ import logging # localization import gettext try: - gettext.install("msec", unicode=1) + gettext.install("msec", str=1) except IOError: _ = str @@ -120,21 +120,21 @@ class BackgroundRunner(Thread): def run(self): """Installs tomoyo policy""" - print "Running %s" % self.program + print("Running %s" % self.program) try: res = os.system(self.program) self.finish.put(res) except: - print "Aborted: %s" % sys.exc_value + print("Aborted: %s" % sys.exc_info()[1]) self.finish.put(-1) class MsecGui: """Msec GUI""" # common columns - (COLUMN_LEVEL, COLUMN_LEVEL_DESCR, COLUMN_LEVEL_CURRENT) = range(3) - (COLUMN_OPTION, COLUMN_DESCR, COLUMN_VALUE, COLUMN_CUSTOM) = range(4) - (COLUMN_PATH, COLUMN_USER, COLUMN_GROUP, COLUMN_PERM, COLUMN_FORCE, COLUMN_ACL) = range(6) - (COLUMN_EXCEPTION, COLUMN_EXCEPTION_VALUE, COLUMN_POS) = range(3) + (COLUMN_LEVEL, COLUMN_LEVEL_DESCR, COLUMN_LEVEL_CURRENT) = list(range(3)) + (COLUMN_OPTION, COLUMN_DESCR, COLUMN_VALUE, COLUMN_CUSTOM) = list(range(4)) + (COLUMN_PATH, COLUMN_USER, COLUMN_GROUP, COLUMN_PERM, COLUMN_FORCE, COLUMN_ACL) = list(range(6)) + (COLUMN_EXCEPTION, COLUMN_EXCEPTION_VALUE, COLUMN_POS) = list(range(3)) def __init__(self, log, msec, perms, msecconfig, permconfig, exceptions, embed=None): """Initializes gui""" @@ -239,7 +239,7 @@ class MsecGui: banner.pack_start(label, False, False, 0) main_vbox.pack_start(banner, False, False, 0) except: - print "Banner %s Not found" % ("%s/%s" % (config.MSEC_DIR, BANNER)) + print("Banner %s Not found" % ("%s/%s" % (config.MSEC_DIR, BANNER))) # creating main UI self.main_notebook = Gtk.Notebook() @@ -271,7 +271,7 @@ class MsecGui: self.toggle_level(self.base_level) # pending signals - self.signals = Queue.Queue() + self.signals = queue.Queue() GLib.timeout_add(500, self.check_signals) self.window.show_all() @@ -280,7 +280,7 @@ class MsecGui: """Switches to a new security level""" iter = model.get_iter(path) level = model.get_value(iter, self.COLUMN_LEVEL) - print "Switching to %s" % level + print("Switching to %s" % level) self.toggle_level(level, force=True) def check_signals(self): @@ -524,7 +524,7 @@ class MsecGui: for option in options: # retreiving option description - if not config.SETTINGS.has_key(option): + if option not in config.SETTINGS: # invalid option self.log.error(_("Invalid option '%s'!") % option) continue @@ -585,7 +585,7 @@ class MsecGui: banner.pack_start(image, False, False, 0) table.attach(banner, 0, 1, row, row+1, Gtk.AttachOptions.EXPAND | Gtk.AttachOptions.FILL, 0, 0, 0) except: - print "Unable to load icon %s: %s" % (icon, sys.exc_value) + print("Unable to load icon %s: %s" % (icon, sys.exc_info()[1])) label = Gtk.Label(label=text) label.set_property("xalign", 0.0) label.modify_font(Pango.FontDescription("12")) @@ -693,7 +693,7 @@ class MsecGui: with open(logfile, "r") as fd: data = fd.readlines() except: - data = [_("Unable to read log file: %s") % Narg(sys.exc_value)] + data = [_("Unable to read log file: %s") % Narg(sys.exc_info()[1])] dialog = Gtk.Dialog(_("Periodic check results"), self.window, 0, @@ -753,7 +753,7 @@ class MsecGui: self.process_events() # queue to signal that job is finished - q = Queue.Queue() + q = queue.Queue() if check == "manual": program = "/usr/share/msec/security.sh" @@ -1741,13 +1741,13 @@ class MsecGui: # {{{ usage def usage(): """Prints help message""" - print """Msec: Mandriva Security Center (%s). + print("""Msec: Mandriva Security Center (%s). Arguments to msecgui: -h, --help displays this helpful message. -d enable debugging messages. -e, --embedded embed in MCC. -""" % version +""" % version) # }}} if __name__ == "__main__": @@ -1770,9 +1770,9 @@ if __name__ == "__main__": log_level = logging.DEBUG elif o[0] == '-e' or o[0] == '--embedded': try: - PlugWindowID = long(o[1]) + PlugWindowID = int(o[1]) except: - print >>sys.stderr, "Error: bad master window XID (%s)!" % o[1] + print("Error: bad master window XID (%s)!" % o[1], file=sys.stderr) sys.exit(1) # configuring logging diff --git a/src/msec/msecperms.py b/src/msec/msecperms.py index a52315a..73a3533 100755 --- a/src/msec/msecperms.py +++ b/src/msec/msecperms.py @@ -18,7 +18,7 @@ import getopt import gettext try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str @@ -37,7 +37,7 @@ from libmsec import Log, PERMS # {{{ usage def usage(): """Prints help message""" - print """Msec: Mageia Security Center (%s). + print("""Msec: Mageia Security Center (%s). This applications verifies and (when required) enforces permissions of certain files and directories. @@ -64,7 +64,7 @@ Available parameters: will perform. -r, --root path to use as root -q, --quiet run quietly -""" % (version, config.PERMCONF, config.PERMCONF) +""" % (version, config.PERMCONF, config.PERMCONF)) # }}} if __name__ == "__main__": @@ -93,13 +93,13 @@ if __name__ == "__main__": permconf = config.load_default_perms(log, level) params = permconf.list_options() if not params: - print >>sys.stderr, _("Invalid security level '%s'.") % level + print(_("Invalid security level '%s'.") % level, file=sys.stderr) sys.exit(1) for file in params: user, group, perm, force = permconf.get(file) if force: - print "!! forcing permissions on %s" % file - print "%s: %s.%s perm %s" % (file, user, group, perm) + print("!! forcing permissions on %s" % file) + print("%s: %s.%s perm %s" % (file, user, group, perm)) sys.exit(0) # debugging elif o[0] == '-d' or o[0] == '--debug': @@ -118,9 +118,9 @@ if __name__ == "__main__": # verifying use id if os.geteuid() != 0: - print >>sys.stderr, _("Msec: Mageia Security Center (%s)\n") % version - print >>sys.stderr, _("Error: This application must be executed by root!") - print >>sys.stderr, _("Run with --help to get help.") + print(_("Msec: Mageia Security Center (%s)\n") % version, file=sys.stderr) + print(_("Error: This application must be executed by root!"), file=sys.stderr) + print(_("Run with --help to get help."), file=sys.stderr) sys.exit(1) # configuring logging diff --git a/src/msec/plugins/audit.py b/src/msec/plugins/audit.py index 2a68aa2..cbc82cc 100755 --- a/src/msec/plugins/audit.py +++ b/src/msec/plugins/audit.py @@ -10,7 +10,7 @@ import config import gettext # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str diff --git a/src/msec/plugins/log.py b/src/msec/plugins/log.py index b7465c7..06d210f 100755 --- a/src/msec/plugins/log.py +++ b/src/msec/plugins/log.py @@ -12,7 +12,7 @@ import config # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str diff --git a/src/msec/plugins/msec.py b/src/msec/plugins/msec.py index 16deef2..554574c 100755 --- a/src/msec/plugins/msec.py +++ b/src/msec/plugins/msec.py @@ -17,7 +17,7 @@ import config import gettext # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str @@ -434,12 +434,12 @@ class msec: liloconf = self.configfiles.get_config_file(LILOCONF) liloconf.exists() and (liloconf.replace_line_matching('^password=', 'password="' + value + '"', 0, 1) or \ liloconf.insert_after('^boot=', 'password="' + value + '"')) and \ - Perms.chmod(liloconf.path, 0600) + Perms.chmod(liloconf.path, 0o600) # TODO encrypt password in grub menulst = self.configfiles.get_config_file(MENULST) menulst.exists() and (menulst.replace_line_matching('^password\s', 'password "' + value + '"') or \ menulst.insert_at(0, 'password "' + value + '"')) and \ - Perms.chmod(menulst.path, 0600) + Perms.chmod(menulst.path, 0o600) # TODO add yaboot support def nopassword_loader(self): diff --git a/src/msec/plugins/network.py b/src/msec/plugins/network.py index cbee32d..307ea87 100755 --- a/src/msec/plugins/network.py +++ b/src/msec/plugins/network.py @@ -17,7 +17,7 @@ import config import gettext # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str diff --git a/src/msec/plugins/pam.py b/src/msec/plugins/pam.py index 353fa2a..c534d5a 100755 --- a/src/msec/plugins/pam.py +++ b/src/msec/plugins/pam.py @@ -14,7 +14,7 @@ import config # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str diff --git a/src/msec/plugins/sectool.py b/src/msec/plugins/sectool.py index 392c5c3..62dc341 100755 --- a/src/msec/plugins/sectool.py +++ b/src/msec/plugins/sectool.py @@ -14,7 +14,7 @@ import config # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str diff --git a/src/msec/plugins/sudo.py b/src/msec/plugins/sudo.py index b61a078..6ab68d8 100755 --- a/src/msec/plugins/sudo.py +++ b/src/msec/plugins/sudo.py @@ -14,7 +14,7 @@ import config # localization try: - gettext.install('msec', unicode=1) + gettext.install('msec', str=1) except IOError: _ = str diff --git a/src/msec/tools.py b/src/msec/tools.py index cb9d1bf..61e3846 100755 --- a/src/msec/tools.py +++ b/src/msec/tools.py @@ -14,7 +14,7 @@ from config import Narg, SYS_ENC # localization import gettext try: - gettext.install("msec", unicode=1) + gettext.install("msec", str=1) except IOError: _ = str @@ -32,7 +32,7 @@ def find_firewall_info(log): if l[:3] == "-A ": firewall_entries.append(l.strip()) except: - log.error(_("Unable to parse firewall configuration: %s") % Narg(sys.exc_value)) + log.error(_("Unable to parse firewall configuration: %s") % Narg(sys.exc_info()[1])) # not find out if the firewall is enabled if len(firewall_entries) == 0: firewall_status = _("Disabled") @@ -48,7 +48,7 @@ def get_updates_status(log, updatedir="/var/lib/urpmi"): updated = time.localtime(ret[stat.ST_MTIME]) status = _("Last updated: %s") % Narg(updated_s) except: - log.error(_("Unable to access %s: %s") % (updatedir, Narg(sys.exc_value))) + log.error(_("Unable to access %s: %s") % (updatedir, Narg(sys.exc_info()[1]))) status = _("Unable to determine update status") return status -- cgit v1.2.1