/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to mandos

  • Committer: Teddy Hogeborn
  • Date: 2019-03-05 21:39:15 UTC
  • Revision ID: teddy@recompile.se-20190305213915-xm1vw00jyy3a5tfn
mandos-ctl: Add more tests, including tests for all commands

* mandos-ctl (Test_string_to_delta.test_handles_basic_rfc3339): Add a
                                                  few more test cases.
  (TestCmd.setUp.MockClient.Set, TestCmd.setUp.MockClient.Get): Don't
  append to self.calls, since nobody should use it to check for Set()
  or Get() calls; instead, the return value of Get() should be
  inspected, and the MockClient.attributes dict should be inspected
  after (implicitly) calling Set().
  (Unique): New; stand-in for unittest.mock.sentinel.
  (TestPropertyCmd): New; abstract class testing PropertyCmd classes.
  (TestBumpTimeoutCmd, TestStartCheckerCmd, TestStopCheckerCmd,
  TestApproveByDefaultCmd, TestDenyByDefaultCmd): New.
  (TestValueArgumentPropertyCmd): New; abstract class for testing
                                  those PropertyCmd classes which also
                                  inherit from ValueArgumentMixIn.
  (TestSetCheckerCmd, TestSetHostCmd, TestSetSecretCmd,
  TestSetTimeoutCmd, TestSetExtendedTimeoutCmd, TestSetIntervalCmd,
  TestSetApprovalDelayCmd, TestSetApprovalDurationCmd): New.

Show diffs side-by-side

added added

removed removed

Lines of Context:
mandos
1
 
#!/usr/bin/python3 -bI
2
 
# -*- mode: python; after-save-hook: (lambda () (let ((command (if (fboundp 'file-local-name) (file-local-name (buffer-file-name)) (or (file-remote-p (buffer-file-name) 'localname) (buffer-file-name))))) (if (= (progn (if (get-buffer "*Test*") (kill-buffer "*Test*")) (process-file-shell-command (format "%s --check" (shell-quote-argument command)) nil "*Test*")) 0) (let ((w (get-buffer-window "*Test*"))) (if w (delete-window w))) (progn (with-current-buffer "*Test*" (compilation-mode)) (display-buffer "*Test*" '(display-buffer-in-side-window)))))); coding: utf-8 -*-
 
1
#!/usr/bin/python
 
2
# -*- mode: python; coding: utf-8 -*-
3
3
#
4
4
# Mandos server - give out binary blobs to connecting clients.
5
5
#
77
77
import itertools
78
78
import collections
79
79
import codecs
80
 
import unittest
81
 
import random
82
 
import shlex
83
80
 
84
81
import dbus
85
82
import dbus.service
86
 
import gi
87
83
from gi.repository import GLib
88
84
from dbus.mainloop.glib import DBusGMainLoop
89
85
import ctypes
91
87
import xml.dom.minidom
92
88
import inspect
93
89
 
94
 
if sys.version_info.major == 2:
95
 
    __metaclass__ = type
96
 
    str = unicode
97
 
 
98
 
# Add collections.abc.Callable if it does not exist
99
 
try:
100
 
    collections.abc.Callable
101
 
except AttributeError:
102
 
    class abc:
103
 
        Callable = collections.Callable
104
 
    collections.abc = abc
105
 
    del abc
106
 
 
107
 
# Add shlex.quote if it does not exist
108
 
try:
109
 
    shlex.quote
110
 
except AttributeError:
111
 
    shlex.quote = re.escape
112
 
 
113
 
# Show warnings by default
114
 
if not sys.warnoptions:
115
 
    import warnings
116
 
    warnings.simplefilter("default")
117
 
 
118
90
# Try to find the value of SO_BINDTODEVICE:
119
91
try:
120
92
    # This is where SO_BINDTODEVICE is in Python 3.3 (or 3.4?) and
140
112
            # No value found
141
113
            SO_BINDTODEVICE = None
142
114
 
143
 
if sys.version_info < (3, 2):
144
 
    configparser.Configparser = configparser.SafeConfigParser
 
115
if sys.version_info.major == 2:
 
116
    str = unicode
145
117
 
146
 
version = "1.8.11"
 
118
version = "1.8.3"
147
119
stored_state_file = "clients.pickle"
148
120
 
149
121
logger = logging.getLogger()
150
 
logging.captureWarnings(True)   # Show warnings via the logging system
151
122
syslogger = None
152
123
 
153
124
try:
208
179
    pass
209
180
 
210
181
 
211
 
class PGPEngine:
 
182
class PGPEngine(object):
212
183
    """A simple class for OpenPGP symmetric encryption & decryption"""
213
184
 
214
185
    def __init__(self):
218
189
            output = subprocess.check_output(["gpgconf"])
219
190
            for line in output.splitlines():
220
191
                name, text, path = line.split(b":")
221
 
                if name == b"gpg":
 
192
                if name == "gpg":
222
193
                    self.gpg = path
223
194
                    break
224
195
        except OSError as e:
229
200
                          '--force-mdc',
230
201
                          '--quiet']
231
202
        # Only GPG version 1 has the --no-use-agent option.
232
 
        if self.gpg == b"gpg" or self.gpg.endswith(b"/gpg"):
 
203
        if self.gpg == "gpg" or self.gpg.endswith("/gpg"):
233
204
            self.gnupgargs.append("--no-use-agent")
234
205
 
235
206
    def __enter__(self):
304
275
 
305
276
 
306
277
# Pretend that we have an Avahi module
307
 
class avahi:
308
 
    """This isn't so much a class as it is a module-like namespace."""
 
278
class Avahi(object):
 
279
    """This isn't so much a class as it is a module-like namespace.
 
280
    It is instantiated once, and simulates having an Avahi module."""
309
281
    IF_UNSPEC = -1               # avahi-common/address.h
310
282
    PROTO_UNSPEC = -1            # avahi-common/address.h
311
283
    PROTO_INET = 0               # avahi-common/address.h
315
287
    DBUS_INTERFACE_SERVER = DBUS_NAME + ".Server"
316
288
    DBUS_PATH_SERVER = "/"
317
289
 
318
 
    @staticmethod
319
 
    def string_array_to_txt_array(t):
 
290
    def string_array_to_txt_array(self, t):
320
291
        return dbus.Array((dbus.ByteArray(s.encode("utf-8"))
321
292
                           for s in t), signature="ay")
322
293
    ENTRY_GROUP_ESTABLISHED = 2  # avahi-common/defs.h
327
298
    SERVER_RUNNING = 2           # avahi-common/defs.h
328
299
    SERVER_COLLISION = 3         # avahi-common/defs.h
329
300
    SERVER_FAILURE = 4           # avahi-common/defs.h
 
301
avahi = Avahi()
330
302
 
331
303
 
332
304
class AvahiError(Exception):
344
316
    pass
345
317
 
346
318
 
347
 
class AvahiService:
 
319
class AvahiService(object):
348
320
    """An Avahi (Zeroconf) service.
349
321
 
350
322
    Attributes:
532
504
 
533
505
 
534
506
# Pretend that we have a GnuTLS module
535
 
class gnutls:
536
 
    """This isn't so much a class as it is a module-like namespace."""
 
507
class GnuTLS(object):
 
508
    """This isn't so much a class as it is a module-like namespace.
 
509
    It is instantiated once, and simulates having a GnuTLS module."""
537
510
 
538
511
    library = ctypes.util.find_library("gnutls")
539
512
    if library is None:
540
513
        library = ctypes.util.find_library("gnutls-deb0")
541
514
    _library = ctypes.cdll.LoadLibrary(library)
542
515
    del library
 
516
    _need_version = b"3.3.0"
 
517
    _tls_rawpk_version = b"3.6.6"
 
518
 
 
519
    def __init__(self):
 
520
        # Need to use "self" here, since this method is called before
 
521
        # the assignment to the "gnutls" global variable happens.
 
522
        if self.check_version(self._need_version) is None:
 
523
            raise self.Error("Needs GnuTLS {} or later"
 
524
                             .format(self._need_version))
543
525
 
544
526
    # Unless otherwise indicated, the constants and types below are
545
527
    # all from the gnutls/gnutls.h C header file.
587
569
 
588
570
    # Exceptions
589
571
    class Error(Exception):
 
572
        # We need to use the class name "GnuTLS" here, since this
 
573
        # exception might be raised from within GnuTLS.__init__,
 
574
        # which is called before the assignment to the "gnutls"
 
575
        # global variable has happened.
590
576
        def __init__(self, message=None, code=None, args=()):
591
577
            # Default usage is by a message string, but if a return
592
578
            # code is passed, convert it to a string with
593
579
            # gnutls.strerror()
594
580
            self.code = code
595
581
            if message is None and code is not None:
596
 
                message = gnutls.strerror(code)
597
 
            return super(gnutls.Error, self).__init__(
 
582
                message = GnuTLS.strerror(code)
 
583
            return super(GnuTLS.Error, self).__init__(
598
584
                message, *args)
599
585
 
600
586
    class CertificateSecurityError(Error):
601
587
        pass
602
588
 
603
589
    # Classes
604
 
    class Credentials:
 
590
    class Credentials(object):
605
591
        def __init__(self):
606
592
            self._c_object = gnutls.certificate_credentials_t()
607
593
            gnutls.certificate_allocate_credentials(
611
597
        def __del__(self):
612
598
            gnutls.certificate_free_credentials(self._c_object)
613
599
 
614
 
    class ClientSession:
 
600
    class ClientSession(object):
615
601
        def __init__(self, socket, credentials=None):
616
602
            self._c_object = gnutls.session_t()
617
603
            gnutls_flags = gnutls.CLIENT
618
 
            if gnutls.check_version(b"3.5.6"):
 
604
            if gnutls.check_version("3.5.6"):
619
605
                gnutls_flags |= gnutls.NO_TICKETS
620
606
            if gnutls.has_rawpk:
621
607
                gnutls_flags |= gnutls.ENABLE_RAWPK
758
744
    check_version.argtypes = [ctypes.c_char_p]
759
745
    check_version.restype = ctypes.c_char_p
760
746
 
761
 
    _need_version = b"3.3.0"
762
 
    if check_version(_need_version) is None:
763
 
        raise self.Error("Needs GnuTLS {} or later"
764
 
                         .format(_need_version))
765
 
 
766
 
    _tls_rawpk_version = b"3.6.6"
767
747
    has_rawpk = bool(check_version(_tls_rawpk_version))
768
748
 
769
749
    if has_rawpk:
823
803
                                                    ctypes.c_size_t)]
824
804
        openpgp_crt_get_fingerprint.restype = _error_code
825
805
 
826
 
    if check_version(b"3.6.4"):
 
806
    if check_version("3.6.4"):
827
807
        certificate_type_get2 = _library.gnutls_certificate_type_get2
828
808
        certificate_type_get2.argtypes = [session_t, ctypes.c_int]
829
809
        certificate_type_get2.restype = _error_code
830
810
 
831
811
    # Remove non-public functions
832
812
    del _error_code, _retry_on_error
 
813
# Create the global "gnutls" object, simulating a module
 
814
gnutls = GnuTLS()
833
815
 
834
816
 
835
817
def call_pipe(connection,       # : multiprocessing.Connection
843
825
    connection.close()
844
826
 
845
827
 
846
 
class Client:
 
828
class Client(object):
847
829
    """A representation of a client host served by this server.
848
830
 
849
831
    Attributes:
850
832
    approved:   bool(); 'None' if not yet approved/disapproved
851
833
    approval_delay: datetime.timedelta(); Time to wait for approval
852
834
    approval_duration: datetime.timedelta(); Duration of one approval
853
 
    checker: multiprocessing.Process(); a running checker process used
854
 
             to see if the client lives. 'None' if no process is
855
 
             running.
 
835
    checker:    subprocess.Popen(); a running checker process used
 
836
                                    to see if the client lives.
 
837
                                    'None' if no process is running.
856
838
    checker_callback_tag: a GLib event source tag, or None
857
839
    checker_command: string; External command which is run to check
858
840
                     if client lives.  %() expansions are done at
1052
1034
        if self.checker_initiator_tag is not None:
1053
1035
            GLib.source_remove(self.checker_initiator_tag)
1054
1036
        self.checker_initiator_tag = GLib.timeout_add(
1055
 
            random.randrange(int(self.interval.total_seconds() * 1000
1056
 
                                 + 1)),
 
1037
            int(self.interval.total_seconds() * 1000),
1057
1038
            self.start_checker)
1058
1039
        # Schedule a disable() when 'timeout' has passed
1059
1040
        if self.disable_initiator_tag is not None:
1066
1047
    def checker_callback(self, source, condition, connection,
1067
1048
                         command):
1068
1049
        """The checker has completed, so take appropriate actions."""
 
1050
        self.checker_callback_tag = None
 
1051
        self.checker = None
1069
1052
        # Read return code from connection (see call_pipe)
1070
1053
        returncode = connection.recv()
1071
1054
        connection.close()
1072
 
        if self.checker is not None:
1073
 
            self.checker.join()
1074
 
        self.checker_callback_tag = None
1075
 
        self.checker = None
1076
1055
 
1077
1056
        if returncode >= 0:
1078
1057
            self.last_checker_status = returncode
1134
1113
        if self.checker is None:
1135
1114
            # Escape attributes for the shell
1136
1115
            escaped_attrs = {
1137
 
                attr: shlex.quote(str(getattr(self, attr)))
 
1116
                attr: re.escape(str(getattr(self, attr)))
1138
1117
                for attr in self.runtime_expansions}
1139
1118
            try:
1140
1119
                command = self.checker_command % escaped_attrs
1167
1146
                kwargs=popen_args)
1168
1147
            self.checker.start()
1169
1148
            self.checker_callback_tag = GLib.io_add_watch(
1170
 
                GLib.IOChannel.unix_new(pipe[0].fileno()),
1171
 
                GLib.PRIORITY_DEFAULT, GLib.IO_IN,
 
1149
                pipe[0].fileno(), GLib.IO_IN,
1172
1150
                self.checker_callback, pipe[0], command)
1173
1151
        # Re-run this periodically if run by GLib.timeout_add
1174
1152
        return True
1429
1407
                raise ValueError("Byte arrays not supported for non-"
1430
1408
                                 "'ay' signature {!r}"
1431
1409
                                 .format(prop._dbus_signature))
1432
 
            value = dbus.ByteArray(bytes(value))
 
1410
            value = dbus.ByteArray(b''.join(chr(byte)
 
1411
                                            for byte in value))
1433
1412
        prop(value)
1434
1413
 
1435
1414
    @dbus.service.method(dbus.PROPERTIES_IFACE,
2240
2219
    del _interface
2241
2220
 
2242
2221
 
2243
 
class ProxyClient:
 
2222
class ProxyClient(object):
2244
2223
    def __init__(self, child_pipe, key_id, fpr, address):
2245
2224
        self._pipe = child_pipe
2246
2225
        self._pipe.send(('init', key_id, fpr, address))
2319
2298
            approval_required = False
2320
2299
            try:
2321
2300
                if gnutls.has_rawpk:
2322
 
                    fpr = b""
 
2301
                    fpr = ""
2323
2302
                    try:
2324
2303
                        key_id = self.key_id(
2325
2304
                            self.peer_certificate(session))
2329
2308
                    logger.debug("Key ID: %s", key_id)
2330
2309
 
2331
2310
                else:
2332
 
                    key_id = b""
 
2311
                    key_id = ""
2333
2312
                    try:
2334
2313
                        fpr = self.fingerprint(
2335
2314
                            self.peer_certificate(session))
2519
2498
        return hex_fpr
2520
2499
 
2521
2500
 
2522
 
class MultiprocessingMixIn:
 
2501
class MultiprocessingMixIn(object):
2523
2502
    """Like socketserver.ThreadingMixIn, but with multiprocessing"""
2524
2503
 
2525
2504
    def sub_process_main(self, request, address):
2537
2516
        return proc
2538
2517
 
2539
2518
 
2540
 
class MultiprocessingMixInWithPipe(MultiprocessingMixIn):
 
2519
class MultiprocessingMixInWithPipe(MultiprocessingMixIn, object):
2541
2520
    """ adds a pipe to the MixIn """
2542
2521
 
2543
2522
    def process_request(self, request, client_address):
2558
2537
 
2559
2538
 
2560
2539
class IPv6_TCPServer(MultiprocessingMixInWithPipe,
2561
 
                     socketserver.TCPServer):
 
2540
                     socketserver.TCPServer, object):
2562
2541
    """IPv6-capable TCP server.  Accepts 'None' as address and/or port
2563
2542
 
2564
2543
    Attributes:
2637
2616
                    raise
2638
2617
        # Only bind(2) the socket if we really need to.
2639
2618
        if self.server_address[0] or self.server_address[1]:
2640
 
            if self.server_address[1]:
2641
 
                self.allow_reuse_address = True
2642
2619
            if not self.server_address[0]:
2643
2620
                if self.address_family == socket.AF_INET6:
2644
2621
                    any_address = "::"  # in6addr_any
2697
2674
    def add_pipe(self, parent_pipe, proc):
2698
2675
        # Call "handle_ipc" for both data and EOF events
2699
2676
        GLib.io_add_watch(
2700
 
            GLib.IOChannel.unix_new(parent_pipe.fileno()),
2701
 
            GLib.PRIORITY_DEFAULT, GLib.IO_IN | GLib.IO_HUP,
 
2677
            parent_pipe.fileno(),
 
2678
            GLib.IO_IN | GLib.IO_HUP,
2702
2679
            functools.partial(self.handle_ipc,
2703
2680
                              parent_pipe=parent_pipe,
2704
2681
                              proc=proc))
2742
2719
                return False
2743
2720
 
2744
2721
            GLib.io_add_watch(
2745
 
                GLib.IOChannel.unix_new(parent_pipe.fileno()),
2746
 
                GLib.PRIORITY_DEFAULT, GLib.IO_IN | GLib.IO_HUP,
 
2722
                parent_pipe.fileno(),
 
2723
                GLib.IO_IN | GLib.IO_HUP,
2747
2724
                functools.partial(self.handle_ipc,
2748
2725
                                  parent_pipe=parent_pipe,
2749
2726
                                  proc=proc,
2764
2741
        if command == 'getattr':
2765
2742
            attrname = request[1]
2766
2743
            if isinstance(client_object.__getattribute__(attrname),
2767
 
                          collections.abc.Callable):
 
2744
                          collections.Callable):
2768
2745
                parent_pipe.send(('function', ))
2769
2746
            else:
2770
2747
                parent_pipe.send((
2781
2758
def rfc3339_duration_to_delta(duration):
2782
2759
    """Parse an RFC 3339 "duration" and return a datetime.timedelta
2783
2760
 
2784
 
    >>> rfc3339_duration_to_delta("P7D") == datetime.timedelta(7)
2785
 
    True
2786
 
    >>> rfc3339_duration_to_delta("PT60S") == datetime.timedelta(0, 60)
2787
 
    True
2788
 
    >>> rfc3339_duration_to_delta("PT60M") == datetime.timedelta(0, 3600)
2789
 
    True
2790
 
    >>> rfc3339_duration_to_delta("PT24H") == datetime.timedelta(1)
2791
 
    True
2792
 
    >>> rfc3339_duration_to_delta("P1W") == datetime.timedelta(7)
2793
 
    True
2794
 
    >>> rfc3339_duration_to_delta("PT5M30S") == datetime.timedelta(0, 330)
2795
 
    True
2796
 
    >>> rfc3339_duration_to_delta("P1DT3M20S") == datetime.timedelta(1, 200)
2797
 
    True
 
2761
    >>> rfc3339_duration_to_delta("P7D")
 
2762
    datetime.timedelta(7)
 
2763
    >>> rfc3339_duration_to_delta("PT60S")
 
2764
    datetime.timedelta(0, 60)
 
2765
    >>> rfc3339_duration_to_delta("PT60M")
 
2766
    datetime.timedelta(0, 3600)
 
2767
    >>> rfc3339_duration_to_delta("PT24H")
 
2768
    datetime.timedelta(1)
 
2769
    >>> rfc3339_duration_to_delta("P1W")
 
2770
    datetime.timedelta(7)
 
2771
    >>> rfc3339_duration_to_delta("PT5M30S")
 
2772
    datetime.timedelta(0, 330)
 
2773
    >>> rfc3339_duration_to_delta("P1DT3M20S")
 
2774
    datetime.timedelta(1, 200)
2798
2775
    """
2799
2776
 
2800
2777
    # Parsing an RFC 3339 duration with regular expressions is not
2880
2857
def string_to_delta(interval):
2881
2858
    """Parse a string and return a datetime.timedelta
2882
2859
 
2883
 
    >>> string_to_delta('7d') == datetime.timedelta(7)
2884
 
    True
2885
 
    >>> string_to_delta('60s') == datetime.timedelta(0, 60)
2886
 
    True
2887
 
    >>> string_to_delta('60m') == datetime.timedelta(0, 3600)
2888
 
    True
2889
 
    >>> string_to_delta('24h') == datetime.timedelta(1)
2890
 
    True
2891
 
    >>> string_to_delta('1w') == datetime.timedelta(7)
2892
 
    True
2893
 
    >>> string_to_delta('5m 30s') == datetime.timedelta(0, 330)
2894
 
    True
 
2860
    >>> string_to_delta('7d')
 
2861
    datetime.timedelta(7)
 
2862
    >>> string_to_delta('60s')
 
2863
    datetime.timedelta(0, 60)
 
2864
    >>> string_to_delta('60m')
 
2865
    datetime.timedelta(0, 3600)
 
2866
    >>> string_to_delta('24h')
 
2867
    datetime.timedelta(1)
 
2868
    >>> string_to_delta('1w')
 
2869
    datetime.timedelta(7)
 
2870
    >>> string_to_delta('5m 30s')
 
2871
    datetime.timedelta(0, 330)
2895
2872
    """
2896
2873
 
2897
2874
    try:
2999
2976
 
3000
2977
    options = parser.parse_args()
3001
2978
 
 
2979
    if options.check:
 
2980
        import doctest
 
2981
        fail_count, test_count = doctest.testmod()
 
2982
        sys.exit(os.EX_OK if fail_count == 0 else 1)
 
2983
 
3002
2984
    # Default values for config file for server-global settings
3003
2985
    if gnutls.has_rawpk:
3004
2986
        priority = ("SECURE128:!CTYPE-X.509:+CTYPE-RAWPK:!RSA"
3024
3006
    del priority
3025
3007
 
3026
3008
    # Parse config file for server-global settings
3027
 
    server_config = configparser.ConfigParser(server_defaults)
 
3009
    server_config = configparser.SafeConfigParser(server_defaults)
3028
3010
    del server_defaults
3029
3011
    server_config.read(os.path.join(options.configdir, "mandos.conf"))
3030
 
    # Convert the ConfigParser object to a dict
 
3012
    # Convert the SafeConfigParser object to a dict
3031
3013
    server_settings = server_config.defaults()
3032
3014
    # Use the appropriate methods on the non-string config options
3033
3015
    for option in ("debug", "use_dbus", "use_ipv6", "restore",
3105
3087
                                  server_settings["servicename"])))
3106
3088
 
3107
3089
    # Parse config file with clients
3108
 
    client_config = configparser.ConfigParser(Client.client_defaults)
 
3090
    client_config = configparser.SafeConfigParser(Client
 
3091
                                                  .client_defaults)
3109
3092
    client_config.read(os.path.join(server_settings["configdir"],
3110
3093
                                    "clients.conf"))
3111
3094
 
3182
3165
        # Close all input and output, do double fork, etc.
3183
3166
        daemon()
3184
3167
 
3185
 
    if gi.version_info < (3, 10, 2):
3186
 
        # multiprocessing will use threads, so before we use GLib we
3187
 
        # need to inform GLib that threads will be used.
3188
 
        GLib.threads_init()
 
3168
    # multiprocessing will use threads, so before we use GLib we need
 
3169
    # to inform GLib that threads will be used.
 
3170
    GLib.threads_init()
3189
3171
 
3190
3172
    global main_loop
3191
3173
    # From the Avahi example code
3267
3249
                             if isinstance(s, bytes)
3268
3250
                             else s) for s in
3269
3251
                            value["client_structure"]]
3270
 
                        # .name, .host, and .checker_command
3271
 
                        for k in ("name", "host", "checker_command"):
 
3252
                        # .name & .host
 
3253
                        for k in ("name", "host"):
3272
3254
                            if isinstance(value[k], bytes):
3273
3255
                                value[k] = value[k].decode("utf-8")
3274
 
                        if "key_id" not in value:
 
3256
                        if not value.has_key("key_id"):
3275
3257
                            value["key_id"] = ""
3276
 
                        elif "fingerprint" not in value:
 
3258
                        elif not value.has_key("fingerprint"):
3277
3259
                            value["fingerprint"] = ""
3278
3260
                    #  old_client_settings
3279
3261
                    # .keys()
3284
3266
                        for key, value in
3285
3267
                        bytes_old_client_settings.items()}
3286
3268
                    del bytes_old_client_settings
3287
 
                    # .host and .checker_command
 
3269
                    # .host
3288
3270
                    for value in old_client_settings.values():
3289
 
                        for attribute in ("host", "checker_command"):
3290
 
                            if isinstance(value[attribute], bytes):
3291
 
                                value[attribute] = (value[attribute]
3292
 
                                                    .decode("utf-8"))
 
3271
                        if isinstance(value["host"], bytes):
 
3272
                            value["host"] = (value["host"]
 
3273
                                             .decode("utf-8"))
3293
3274
            os.remove(stored_state_path)
3294
3275
        except IOError as e:
3295
3276
            if e.errno == errno.ENOENT:
3620
3601
                sys.exit(1)
3621
3602
            # End of Avahi example code
3622
3603
 
3623
 
        GLib.io_add_watch(
3624
 
            GLib.IOChannel.unix_new(tcp_server.fileno()),
3625
 
            GLib.PRIORITY_DEFAULT, GLib.IO_IN,
3626
 
            lambda *args, **kwargs: (tcp_server.handle_request
3627
 
                                     (*args[2:], **kwargs) or True))
 
3604
        GLib.io_add_watch(tcp_server.fileno(), GLib.IO_IN,
 
3605
                          lambda *args, **kwargs:
 
3606
                          (tcp_server.handle_request
 
3607
                           (*args[2:], **kwargs) or True))
3628
3608
 
3629
3609
        logger.debug("Starting main loop")
3630
3610
        main_loop.run()
3640
3620
    # Must run before the D-Bus bus name gets deregistered
3641
3621
    cleanup()
3642
3622
 
3643
 
 
3644
 
def should_only_run_tests():
3645
 
    parser = argparse.ArgumentParser(add_help=False)
3646
 
    parser.add_argument("--check", action='store_true')
3647
 
    args, unknown_args = parser.parse_known_args()
3648
 
    run_tests = args.check
3649
 
    if run_tests:
3650
 
        # Remove --check argument from sys.argv
3651
 
        sys.argv[1:] = unknown_args
3652
 
    return run_tests
3653
 
 
3654
 
# Add all tests from doctest strings
3655
 
def load_tests(loader, tests, none):
3656
 
    import doctest
3657
 
    tests.addTests(doctest.DocTestSuite())
3658
 
    return tests
3659
3623
 
3660
3624
if __name__ == '__main__':
3661
 
    try:
3662
 
        if should_only_run_tests():
3663
 
            # Call using ./mandos --check [--verbose]
3664
 
            unittest.main()
3665
 
        else:
3666
 
            main()
3667
 
    finally:
3668
 
        logging.shutdown()
 
3625
    main()