/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-ctl

  • Committer: Teddy Hogeborn
  • Date: 2016-03-17 20:40:55 UTC
  • Revision ID: teddy@recompile.se-20160317204055-bhsh5xsidq7w5cxu
Client: Fix plymouth agent; broken since 1.7.2.

Fix an very old memory bug in the plymouth agent (which has been
present since its apperance in version 1.2), but which was only
recently detected at run time due to the new -fsanitize=address
compile- time flag, which has been used since version 1.7.2.  This
detection of a memory access violation causes the program to abort,
making the Plymouth graphical boot system unable to accept interactive
input of passwords when using the Mandos client.

* plugins.d/plymouth.c (exec_and_wait): Fix memory allocation bug when
  allocating new_argv.  Also tolerate a zero-length argv.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
1
#!/usr/bin/python
2
2
# -*- mode: python; coding: utf-8 -*-
3
 
#
 
3
4
4
# Mandos Monitor - Control and monitor the Mandos server
5
 
#
6
 
# Copyright © 2008-2017 Teddy Hogeborn
7
 
# Copyright © 2008-2017 Björn Påhlsson
8
 
#
 
5
 
6
# Copyright © 2008-2016 Teddy Hogeborn
 
7
# Copyright © 2008-2016 Björn Påhlsson
 
8
9
9
# This program is free software: you can redistribute it and/or modify
10
10
# it under the terms of the GNU General Public License as published by
11
11
# the Free Software Foundation, either version 3 of the License, or
15
15
#     but WITHOUT ANY WARRANTY; without even the implied warranty of
16
16
#     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
17
17
#     GNU General Public License for more details.
18
 
#
 
18
19
19
# You should have received a copy of the GNU General Public License
20
20
# along with this program.  If not, see
21
21
# <http://www.gnu.org/licenses/>.
22
 
#
 
22
23
23
# Contact the authors at <mandos@recompile.se>.
24
 
#
 
24
25
25
 
26
26
from __future__ import (division, absolute_import, print_function,
27
27
                        unicode_literals)
38
38
import re
39
39
import os
40
40
import collections
41
 
import json
 
41
import doctest
42
42
 
43
43
import dbus
44
44
 
64
64
    "ApprovalDelay": "Approval Delay",
65
65
    "ApprovalDuration": "Approval Duration",
66
66
    "Checker": "Checker",
67
 
    "ExtendedTimeout": "Extended Timeout",
68
 
    "Expires": "Expires",
69
 
    "LastCheckerStatus": "Last Checker Status",
 
67
    "ExtendedTimeout": "Extended Timeout"
70
68
}
71
69
defaultkeywords = ("Name", "Enabled", "Timeout", "LastCheckedOK")
72
70
domain = "se.recompile"
74
72
server_path = "/"
75
73
server_interface = domain + ".Mandos"
76
74
client_interface = domain + ".Mandos.Client"
77
 
version = "1.7.15"
 
75
version = "1.7.6"
78
76
 
79
77
 
80
78
try:
82
80
except AttributeError:
83
81
    dbus.OBJECT_MANAGER_IFACE = "org.freedesktop.DBus.ObjectManager"
84
82
 
85
 
 
86
83
def milliseconds_to_string(ms):
87
84
    td = datetime.timedelta(0, 0, 0, ms)
88
 
    return ("{days}{hours:02}:{minutes:02}:{seconds:02}"
89
 
            .format(days="{}T".format(td.days) if td.days else "",
90
 
                    hours=td.seconds // 3600,
91
 
                    minutes=(td.seconds % 3600) // 60,
92
 
                    seconds=td.seconds % 60))
 
85
    return ("{days}{hours:02}:{minutes:02}:{seconds:02}".format(
 
86
        days = "{}T".format(td.days) if td.days else "",
 
87
        hours = td.seconds // 3600,
 
88
        minutes = (td.seconds % 3600) // 60,
 
89
        seconds = td.seconds % 60))
93
90
 
94
91
 
95
92
def rfc3339_duration_to_delta(duration):
96
93
    """Parse an RFC 3339 "duration" and return a datetime.timedelta
97
 
 
 
94
    
98
95
    >>> rfc3339_duration_to_delta("P7D")
99
96
    datetime.timedelta(7)
100
97
    >>> rfc3339_duration_to_delta("PT60S")
110
107
    >>> rfc3339_duration_to_delta("P1DT3M20S")
111
108
    datetime.timedelta(1, 200)
112
109
    """
113
 
 
 
110
    
114
111
    # Parsing an RFC 3339 duration with regular expressions is not
115
112
    # possible - there would have to be multiple places for the same
116
113
    # values, like seconds.  The current code, while more esoteric, is
117
114
    # cleaner without depending on a parsing library.  If Python had a
118
115
    # built-in library for parsing we would use it, but we'd like to
119
116
    # avoid excessive use of external libraries.
120
 
 
 
117
    
121
118
    # New type for defining tokens, syntax, and semantics all-in-one
122
119
    Token = collections.namedtuple("Token", (
123
120
        "regexp",  # To match token; if "value" is not None, must have
156
153
                           frozenset((token_year, token_month,
157
154
                                      token_day, token_time,
158
155
                                      token_week)))
159
 
    # Define starting values:
160
 
    # Value so far
161
 
    value = datetime.timedelta()
 
156
    # Define starting values
 
157
    value = datetime.timedelta() # Value so far
162
158
    found_token = None
163
 
    # Following valid tokens
164
 
    followers = frozenset((token_duration, ))
165
 
    # String left to parse
166
 
    s = duration
 
159
    followers = frozenset((token_duration, )) # Following valid tokens
 
160
    s = duration                # String left to parse
167
161
    # Loop until end token is found
168
162
    while found_token is not token_end:
169
163
        # Search for any currently valid tokens
193
187
 
194
188
def string_to_delta(interval):
195
189
    """Parse a string and return a datetime.timedelta
196
 
 
 
190
    
197
191
    >>> string_to_delta('7d')
198
192
    datetime.timedelta(7)
199
193
    >>> string_to_delta('60s')
207
201
    >>> string_to_delta('5m 30s')
208
202
    datetime.timedelta(0, 330)
209
203
    """
210
 
 
 
204
    
211
205
    try:
212
206
        return rfc3339_duration_to_delta(interval)
213
207
    except ValueError:
214
208
        pass
215
 
 
 
209
    
216
210
    value = datetime.timedelta(0)
217
211
    regexp = re.compile(r"(\d+)([dsmhw]?)")
218
 
 
 
212
    
219
213
    for num, suffix in regexp.findall(interval):
220
214
        if suffix == "d":
221
215
            value += datetime.timedelta(int(num))
240
234
                       "ApprovalDuration", "ExtendedTimeout"):
241
235
            return milliseconds_to_string(value)
242
236
        return str(value)
243
 
 
 
237
    
244
238
    # Create format string to print table rows
245
239
    format_string = " ".join("{{{key}:{width}}}".format(
246
 
        width=max(len(tablewords[key]),
247
 
                  max(len(valuetostring(client[key], key))
248
 
                      for client in clients)),
249
 
        key=key)
 
240
        width = max(len(tablewords[key]),
 
241
                    max(len(valuetostring(client[key], key))
 
242
                        for client in clients)),
 
243
        key = key)
250
244
                             for key in keywords)
251
245
    # Print header line
252
246
    print(format_string.format(**tablewords))
253
247
    for client in clients:
254
 
        print(format_string
255
 
              .format(**{key: valuetostring(client[key], key)
256
 
                         for key in keywords}))
 
248
        print(format_string.format(**{
 
249
            key: valuetostring(client[key], key)
 
250
            for key in keywords }))
257
251
 
258
252
 
259
253
def has_actions(options):
280
274
def main():
281
275
    parser = argparse.ArgumentParser()
282
276
    parser.add_argument("--version", action="version",
283
 
                        version="%(prog)s {}".format(version),
 
277
                        version = "%(prog)s {}".format(version),
284
278
                        help="show version number and exit")
285
279
    parser.add_argument("-a", "--all", action="store_true",
286
280
                        help="Select all clients")
287
281
    parser.add_argument("-v", "--verbose", action="store_true",
288
282
                        help="Print all fields")
289
 
    parser.add_argument("-j", "--dump-json", action="store_true",
290
 
                        help="Dump client data in JSON format")
291
283
    parser.add_argument("-e", "--enable", action="store_true",
292
284
                        help="Enable client")
293
285
    parser.add_argument("-d", "--disable", action="store_true",
332
324
                        help="Run self-test")
333
325
    parser.add_argument("client", nargs="*", help="Client name")
334
326
    options = parser.parse_args()
335
 
 
 
327
    
336
328
    if has_actions(options) and not (options.client or options.all):
337
329
        parser.error("Options require clients names or --all.")
338
330
    if options.verbose and has_actions(options):
339
 
        parser.error("--verbose can only be used alone.")
340
 
    if options.dump_json and (options.verbose
341
 
                              or has_actions(options)):
342
 
        parser.error("--dump-json can only be used alone.")
 
331
        parser.error("--verbose can only be used alone or with"
 
332
                     " --all.")
343
333
    if options.all and not has_actions(options):
344
334
        parser.error("--all requires an action.")
345
335
 
346
336
    if options.check:
347
 
        import doctest
348
337
        fail_count, test_count = doctest.testmod()
349
338
        sys.exit(os.EX_OK if fail_count == 0 else 1)
350
 
 
 
339
    
351
340
    try:
352
341
        bus = dbus.SystemBus()
353
342
        mandos_dbus_objc = bus.get_object(busname, server_path)
354
343
    except dbus.exceptions.DBusException:
355
344
        print("Could not connect to Mandos server", file=sys.stderr)
356
345
        sys.exit(1)
357
 
 
 
346
    
358
347
    mandos_serv = dbus.Interface(mandos_dbus_objc,
359
 
                                 dbus_interface=server_interface)
 
348
                                 dbus_interface = server_interface)
360
349
    mandos_serv_object_manager = dbus.Interface(
361
 
        mandos_dbus_objc, dbus_interface=dbus.OBJECT_MANAGER_IFACE)
362
 
 
363
 
    # block stderr since dbus library prints to stderr
 
350
        mandos_dbus_objc, dbus_interface = dbus.OBJECT_MANAGER_IFACE)
 
351
    
 
352
    #block stderr since dbus library prints to stderr
364
353
    null = os.open(os.path.devnull, os.O_RDWR)
365
354
    stderrcopy = os.dup(sys.stderr.fileno())
366
355
    os.dup2(null, sys.stderr.fileno())
367
356
    os.close(null)
368
357
    try:
369
358
        try:
370
 
            mandos_clients = {path: ifs_and_props[client_interface]
371
 
                              for path, ifs_and_props in
372
 
                              mandos_serv_object_manager
373
 
                              .GetManagedObjects().items()
374
 
                              if client_interface in ifs_and_props}
 
359
            mandos_clients = { path: ifs_and_props[client_interface]
 
360
                               for path, ifs_and_props in
 
361
                               mandos_serv_object_manager
 
362
                               .GetManagedObjects().items()
 
363
                               if client_interface in ifs_and_props }
375
364
        finally:
376
 
            # restore stderr
 
365
            #restore stderr
377
366
            os.dup2(stderrcopy, sys.stderr.fileno())
378
367
            os.close(stderrcopy)
379
368
    except dbus.exceptions.DBusException as e:
380
 
        print("Access denied: "
381
 
              "Accessing mandos server through D-Bus: {}".format(e),
382
 
              file=sys.stderr)
 
369
        print("Access denied: Accessing mandos server through D-Bus: {}"
 
370
              .format(e), file=sys.stderr)
383
371
        sys.exit(1)
384
 
 
 
372
    
385
373
    # Compile dict of (clients: properties) to process
386
 
    clients = {}
387
 
 
 
374
    clients={}
 
375
    
388
376
    if options.all or not options.client:
389
 
        clients = {bus.get_object(busname, path): properties
390
 
                   for path, properties in mandos_clients.items()}
 
377
        clients = { bus.get_object(busname, path): properties
 
378
                    for path, properties in mandos_clients.items() }
391
379
    else:
392
380
        for name in options.client:
393
381
            for path, client in mandos_clients.items():
399
387
                print("Client not found on server: {!r}"
400
388
                      .format(name), file=sys.stderr)
401
389
                sys.exit(1)
402
 
 
 
390
    
403
391
    if not has_actions(options) and clients:
404
 
        if options.verbose or options.dump_json:
 
392
        if options.verbose:
405
393
            keywords = ("Name", "Enabled", "Timeout", "LastCheckedOK",
406
394
                        "Created", "Interval", "Host", "Fingerprint",
407
395
                        "CheckerRunning", "LastEnabled",
408
396
                        "ApprovalPending", "ApprovedByDefault",
409
397
                        "LastApprovalRequest", "ApprovalDelay",
410
398
                        "ApprovalDuration", "Checker",
411
 
                        "ExtendedTimeout", "Expires",
412
 
                        "LastCheckerStatus")
 
399
                        "ExtendedTimeout")
413
400
        else:
414
401
            keywords = defaultkeywords
415
 
 
416
 
        if options.dump_json:
417
 
            json.dump({client["Name"]: {key:
418
 
                                        bool(client[key])
419
 
                                        if isinstance(client[key],
420
 
                                                      dbus.Boolean)
421
 
                                        else client[key]
422
 
                                        for key in keywords}
423
 
                       for client in clients.values()},
424
 
                      fp=sys.stdout, indent=4,
425
 
                      separators=(',', ': '))
426
 
            print()
427
 
        else:
428
 
            print_clients(clients.values(), keywords)
 
402
        
 
403
        print_clients(clients.values(), keywords)
429
404
    else:
430
405
        # Process each client in the list by all selected options
431
406
        for client in clients:
432
 
 
 
407
            
433
408
            def set_client_prop(prop, value):
434
409
                """Set a Client D-Bus property"""
435
410
                client.Set(client_interface, prop, value,
436
411
                           dbus_interface=dbus.PROPERTIES_IFACE)
437
 
 
 
412
            
438
413
            def set_client_prop_ms(prop, value):
439
414
                """Set a Client D-Bus property, converted
440
415
                from a string to milliseconds."""
441
416
                set_client_prop(prop,
442
417
                                string_to_delta(value).total_seconds()
443
418
                                * 1000)
444
 
 
 
419
            
445
420
            if options.remove:
446
421
                mandos_serv.RemoveClient(client.__dbus_object_path__)
447
422
            if options.enable:
455
430
            if options.stop_checker:
456
431
                set_client_prop("CheckerRunning", dbus.Boolean(False))
457
432
            if options.is_enabled:
458
 
                if client.Get(client_interface, "Enabled",
459
 
                              dbus_interface=dbus.PROPERTIES_IFACE):
460
 
                    sys.exit(0)
461
 
                else:
462
 
                    sys.exit(1)
 
433
                sys.exit(0 if client.Get(client_interface,
 
434
                                         "Enabled",
 
435
                                         dbus_interface=
 
436
                                         dbus.PROPERTIES_IFACE)
 
437
                         else 1)
463
438
            if options.checker is not None:
464
439
                set_client_prop("Checker", options.checker)
465
440
            if options.host is not None: