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

  • Committer: Teddy Hogeborn
  • Date: 2018-02-18 01:03:48 UTC
  • Revision ID: teddy@recompile.se-20180218010348-hhjfox5t4py5n5zh
Client: Bug fix: Possible memory leak if failing to open /dev/null

* plugins.d/mandos-client.d (run_network_hooks): Try to open /dev/null
  before reading network hook dir in order to fail early in that case.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
1
-*- org -*-
2
2
 
3
 
* Use _attribute_((nonnull)) wherever possible.
 
3
* Testing
 
4
** python-nemu
4
5
 
5
6
* mandos-applet
6
7
 
7
8
* mandos-client
8
 
** TODO [#B] use scandir(3) instead of readdir(3)
9
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
10
 
** TODO [#B] Retry a server which has a non-definite reply:
11
 
*** A closed connection during the TLS handshake
12
 
*** A TCP timeout
 
9
** TODO A --server option which only adds to the server list.
 
10
   (Unlike --connect, which implicitly disables zeroconf.)
13
11
** TODO [#B] Use capabilities instead of seteuid().
14
 
** TODO [#A] Retry --connect forever
 
12
   https://forums.grsecurity.net/viewtopic.php?f=7&t=2522
 
13
** TODO [#B] Use getaddrinfo(hints=AI_NUMERICHOST) instead of inet_pton()
 
14
** TODO [#C] Make start_mandos_communication() take "struct server".
 
15
** TODO [#C] --interfaces=regex,eth*,noregex (bridge-utils-interfaces(5))
 
16
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
15
17
 
16
18
* splashy
17
19
** TODO [#B] use scandir(3) instead of readdir(3)
18
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
20
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
19
21
 
20
 
* usplash
21
 
** TODO [#A] Make it work again
 
22
* usplash (Deprecated)
 
23
** TODO [#B] Make it work again
22
24
** TODO [#B] use scandir(3) instead of readdir(3)
23
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
24
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
 
25
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
25
26
 
26
27
* askpass-fifo
27
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
28
 
** TODO [#B] Drop privileges after opening FIFO.
 
28
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
29
29
 
30
30
* password-prompt
31
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
32
31
** TODO [#B] lock stdin (with flock()?)
 
32
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
33
 
 
34
* plymouth
 
35
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
33
36
 
34
37
* TODO [#B] passdev
35
38
 
36
39
* plugin-runner
37
 
** TODO [#B] use scandir(3) instead of readdir(3)
 
40
** TODO handle printing for errors for plugins
 
41
*** Hook up stderr of plugins, buffer them, and prepend "Mandos Plugin [plugin name]"
38
42
** TODO [#C] use same file name rules as run-parts(8)
39
43
** kernel command line option for debug info
40
 
** TODO [#B] Use openat()
 
44
** TODO [#A] Restart plugins which exit with EX_TEMPFAIL
41
45
 
42
46
* mandos (server)
 
47
** TODO [#B] --notify-command
 
48
   This would allow the mandos.service to use
 
49
   --notify-command="systemd-notify --pid --ready"
 
50
** TODO [#B] python-systemd
 
51
*** import systemd.daemon; systemd.daemon.notify()
43
52
** TODO [#B] Log level                                                    :BUGS:
44
 
** TODO Persistent state                                                  :BUGS:
45
 
   /var/lib/mandos/*
46
53
*** TODO /etc/mandos/clients.d/*.conf
47
54
    Watch this directory and add/remove/update clients?
48
55
** TODO [#C] config for TXT record
49
 
** TODO Log level option
50
 
   syslogger.setLevel(logging.WARNING)
51
 
   + SetLogLevel D-Bus call
52
 
** TODO Implement --foreground                                            :BUGS:
53
 
   [[info:standards:Option%20Table][Table of Long Options]]
54
 
** TODO Implement --socket
55
 
   [[info:standards:Option%20Table][Table of Long Options]]
56
 
** TODO Date+time on console log messages                                 :BUGS:
57
 
   Is this the default?
 
56
** TODO Log level dbus option
 
57
   SetLogLevel D-Bus call
58
58
** TODO [#C] DBusServiceObjectUsingSuper
59
59
** TODO [#B] Global enable/disable flag
60
 
** TODO [#B] By-client countdown on secrets given
61
 
** TODO [#B] Fix problem with fsck taking a really long time
62
 
   Whenever a client successfully gets a secret it could get a
63
 
   one-time timeout boost to allow for an fsck-incurred delay
64
 
** TODO [#A] Delay before client receives key
65
 
   This would give an operator opportunity to cancel the request if
66
 
   desired.
67
 
** TODO [#A] Client manual approval mode
68
 
   A client needs manual approval on the server before it gets the
69
 
   secret
70
 
** TODO [#B] Support RFC 3339 time duration syntax
71
 
** More D-Bus methods
72
 
*** NeedsApproval(50, True) -> timeout, default approve
73
 
    Default approval is configurable, but True by default
74
 
    + Approve(True) -> approve sending saved
75
 
    + Approve(False) -> Close client connection immediately
76
 
*** NeedsPassword(50) - Timeout, default disapprove
 
60
** TODO [#B] By-client countdown on number of secrets given
 
61
** D-Bus Client method NeedsPassword(50) - Timeout, default disapprove
77
62
    + SetPass(u"gazonk", True) -> Approval, persistent
78
63
    + Approve(False) -> Close client connection immediately
79
64
** TODO [#C] python-parsedatetime
80
 
** TODO [#C] systemd/launchd
81
 
   http://0pointer.de/blog/projects/systemd.html
82
65
** TODO Separate logging logic to own object
83
 
** TODO make clients to a dict!
84
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
 
66
** TODO [#B] Limit approval_delay to max gnutls/tls timeout value
85
67
** TODO [#B] break the wait on approval_delay if connection dies
86
68
** TODO Generate Client.runtime_expansions from client options + extra
87
69
** TODO Allow %%(checker)s as a runtime expansion
88
 
 
89
 
* mandos.xml
90
 
** Add mandos contact info in manual pages
 
70
** TODO D-Bus AddClient() method on server object
 
71
** TODO Use org.freedesktop.DBus.Method.NoReply annotation on async methods. :2:
 
72
** TODO Save state periodically to recover better from hard shutdowns
 
73
** TODO CheckerCompleted method, deprecate CheckedOK
 
74
** TODO Secret Service API?
 
75
   https://standards.freedesktop.org/secret-service/
 
76
** TODO Remove D-Bus interfaces with old domain name                      :2:
 
77
** TODO Remove old string_to_delta format                                 :2:
 
78
** TODO http://0pointer.de/blog/projects/stateless.html
 
79
*** File in /usr/lib/sysusers.d to create user+group "_mandos"
 
80
** TODO Error handling on error parsing config files
 
81
** TODO init.d script error handling
 
82
** TODO D-Bus server properties; address, port, interface, etc.           :2:
 
83
** Python 3                                                               :2:
 
84
*** TODO [#C] In Python 3.3, use shlex.quote() instead of re.escape()
91
85
 
92
86
* mandos-ctl
93
87
*** Handle "no D-Bus server" and/or "no Mandos server found" better
94
 
*** [#B] --dump option
95
 
** TODO Support RFC 3339 time duration syntax
 
88
** TODO Remove old string_to_delta format                                 :2:
96
89
 
97
90
* TODO mandos-dispatch
98
91
  Listens for specified D-Bus signals and spawns shell commands with
99
92
  arguments.
100
93
 
101
94
* mandos-monitor
102
 
** TODO help should be toggable
 
95
** TODO --servicename                                                  :BUGS:
 
96
** TODO help should be toggleable
103
97
** Urwid client data displayer
104
98
   Better view of client data in the listing
105
99
*** Properties popup
106
 
** Nicer crashes. Stack traces Messes up shell.
107
 
*** Print a nice "We are sorry" message, save stack trace to log.
108
 
** Show timeout countdown for approval
 
100
** Print a nice "We are sorry" message, save stack trace to log.
109
101
 
110
102
* mandos-keygen
111
 
** TODO Loop until passwords match when run interactively
112
103
** TODO "--secfile" option
113
104
   Using the "secfile" option instead of "secret"
114
105
** TODO [#B] "--test" option
115
106
   For testing decryption before rebooting.
116
107
 
117
 
* Makefile
118
 
** TODO Add "--Xlinker --as-needed"
119
 
   http://udrepper.livejournal.com/19395.html
120
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
121
 
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
122
 
 
123
108
* Package
124
109
** /usr/share/initramfs-tools/hooks/mandos
125
110
*** TODO [#C] use same file name rules as run-parts(8)
126
111
*** TODO [#C] Do not install in initrd.img if configured not to.
127
112
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
128
 
** TODO [#C] /etc/bash_completion.d/mandos
 
113
** TODO [#C] $(pkg-config --variable=completionsdir bash-completion)
129
114
   From XML sources directly?
130
115
 
131
116
* Side Stuff
132
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
133
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
 
117
** TODO Locate which package moves the other bin/sh when busybox is deactivated
 
118
** TODO contact owner of package, and ask them to have that shell static in position regardless of busybox
 
119
 
 
120
* [[http://www.undeadly.org/cgi?action=article&sid=20110530221728][OpenBSD]]
 
121
 
 
122
* TODO Use raw public keys (RFC 7250) for TLS communications              :2:
 
123
** Support for this is planned for GnuTLS version 3.6
 
124
   https://gitlab.com/gnutls/gnutls/issues/26
 
125
** Rationale
 
126
*** The client key is used both for communication and encryption
 
127
    Using raw keys in GnuTLS instead uses separate keys for
 
128
    communication and password decryption.
 
129
*** GnuTLS 3.5.9 has deprecated the OpenPGP functions
 
130
    The functions are still available, but deprecated:
 
131
    https://gitlab.com/gnutls/gnutls/issues/102
134
132
 
135
133
 
136
134
#+STARTUP: showall