/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: 2019-07-30 17:03:57 UTC
  • Revision ID: teddy@recompile.se-20190730170357-jte0piul5mq7j5pr
http://bugs.debian.org/933387
Server: Reap zombies created by multiprocessing.Process()

When creating checkers as multiprocessing.Process() objects, the
multiprocessing module also creates a parent process (for the
call_pipe() function) to call the actual checker process, but this
parent process is not reaped.  This is not a huge problem, since the
zombie is always reaped automatically the next time the multiprocess
starts a new process, but the zombies can be up to as many as there
have ever been simultaneous checker processes.  To fix this, the
process object must be join():ed when they report completion of the
child checker process.

* mandos (Client): Fix doc string to correctly state that
                   Client.checker is a multiprocess.Process() and not
                   a subprocess.Popen() object.
  (Client.checker_callback): After the returncode of the checker
                             process has been read, wait for the
                             self.checker Process object to finish by
                             calling join() on it.

Reported-by: Peter Palfrader <weasel@debian.org>

Show diffs side-by-side

added added

removed removed

Lines of Context:
TODO
1
1
-*- org -*-
2
2
 
3
 
* GIT
4
 
** General: [[https://www.atlassian.com/git/workflows][Git Workflows]], [[http://gitimmersion.com/][Git Immersion]], [[https://news.ycombinator.com/item?id=7036628][Simple git workflow is simple]]
5
 
** Intro: [[http://www.eyrie.org/~eagle/notes/debian/git.html#combine][Using Git for Debian Packaging]]
6
 
** Use: [[https://honk.sigxcpu.org/piki/projects/git-buildpackage/][git-buildpackage]]
7
 
** Migration
8
 
   tailor?
9
 
   Using bzr-fastimport: [[http://www.fusonic.net/en/blog/2013/03/26/migrating-from-bazaar-to-git/][Migrating from Bazaar to Git]]
10
 
** Unresolved: [[http://jameswestby.net/bzr/builddeb/user_manual/split.html][bzr builddeb split mode]]
11
 
   Maybe: [[http://honk.sigxcpu.org/projects/git-buildpackage/manual-html/gbp.import.html#GBP.IMPORT.UPSTREAM.GIT.NOTARBALL][git-buildpackage - No upstream tarballs]]
12
 
   [[http://www.python.org/dev/peps/pep-0374/][PEP 374 - Choosing a distributed VCS for the Python project]]
13
 
   [[http://www.emacswiki.org/emacs/GitForEmacsDevs][Git For Emacs Devs]]
14
 
 
15
 
* [[http://www.undeadly.org/cgi?action=article&sid=20110530221728][OpenBSD]]
16
 
 
17
3
* Testing
18
4
** python-nemu
19
5
 
20
6
* mandos-applet
21
7
 
22
8
* mandos-client
 
9
** TODO A --server option which only adds to the server list.
 
10
   (Unlike --connect, which implicitly disables zeroconf.)
23
11
** TODO [#B] Use capabilities instead of seteuid().
 
12
   https://forums.grsecurity.net/viewtopic.php?f=7&t=2522
24
13
** TODO [#B] Use getaddrinfo(hints=AI_NUMERICHOST) instead of inet_pton()
25
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
 
17
** TODO [#B] Use reallocarray() with GNU LibC 2.29 or later.
26
18
 
27
19
* splashy
28
20
** TODO [#B] use scandir(3) instead of readdir(3)
 
21
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
29
22
 
30
23
* usplash (Deprecated)
31
 
** TODO [#A] Make it work again
 
24
** TODO [#B] Make it work again
32
25
** TODO [#B] use scandir(3) instead of readdir(3)
 
26
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
33
27
 
34
28
* askpass-fifo
 
29
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
35
30
 
36
31
* password-prompt
37
32
** TODO [#B] lock stdin (with flock()?)
 
33
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
38
34
 
39
35
* plymouth
 
36
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
37
** TODO [#B] Use reallocarray() with GNU LibC 2.29 or later.
40
38
 
41
39
* TODO [#B] passdev
42
40
 
45
43
*** Hook up stderr of plugins, buffer them, and prepend "Mandos Plugin [plugin name]"
46
44
** TODO [#C] use same file name rules as run-parts(8)
47
45
** kernel command line option for debug info
 
46
** TODO [#A] Restart plugins which exit with EX_TEMPFAIL
48
47
 
49
48
* mandos (server)
 
49
** TODO [#B] --notify-command
 
50
   This would allow the mandos.service to use
 
51
   --notify-command="systemd-notify --pid --ready"
 
52
** TODO [#B] python-systemd
 
53
*** import systemd.daemon; systemd.daemon.notify()
50
54
** TODO [#B] Log level                                                    :BUGS:
51
55
*** TODO /etc/mandos/clients.d/*.conf
52
56
    Watch this directory and add/remove/update clients?
61
65
    + Approve(False) -> Close client connection immediately
62
66
** TODO [#C] python-parsedatetime
63
67
** TODO Separate logging logic to own object
64
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
 
68
** TODO [#B] Limit approval_delay to max gnutls/tls timeout value
65
69
** TODO [#B] break the wait on approval_delay if connection dies
66
70
** TODO Generate Client.runtime_expansions from client options + extra
67
71
** TODO Allow %%(checker)s as a runtime expansion
68
 
** TODO Use python-tlslite?
69
72
** TODO D-Bus AddClient() method on server object
70
73
** TODO Use org.freedesktop.DBus.Method.NoReply annotation on async methods. :2:
71
 
** TODO Emit [[http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-properties][org.freedesktop.DBus.Properties.PropertiesChanged]] signal            :2:
72
 
   TODO Deprecate se.recompile.Mandos.Client.PropertyChanged - annotate!
73
 
   TODO Can use "invalidates" annotation to also emit on changed secret.
74
 
** TODO Support [[http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-objectmanager][org.freedesktop.DBus.ObjectManager]] interface on server object :2:
75
 
   Deprecate methods GetAllClients(), GetAllClientsWithProperties()
76
 
   and signals ClientAdded and ClientRemoved.
77
74
** TODO Save state periodically to recover better from hard shutdowns
78
75
** TODO CheckerCompleted method, deprecate CheckedOK
79
76
** TODO Secret Service API?
80
 
   http://standards.freedesktop.org/secret-service/
81
 
** TODO Remove D-Bus interfaces with old domain name                         :2:
82
 
** TODO Remove old string_to_delta format                                    :2:
 
77
   https://standards.freedesktop.org/secret-service/
 
78
** TODO Remove D-Bus interfaces with old domain name                      :2:
 
79
** TODO Remove old string_to_delta format                                 :2:
83
80
** TODO http://0pointer.de/blog/projects/stateless.html
84
 
*** tmpfiles snippet to create /var/lib/mandos with right user+perms
85
81
*** File in /usr/lib/sysusers.d to create user+group "_mandos"
86
82
** TODO Error handling on error parsing config files
87
83
** TODO init.d script error handling
88
 
** TODO D-Bus server properties; address, port, interface, etc.              :2:
89
 
 
90
 
* mandos.xml
91
 
** Add mandos contact info in manual pages
 
84
** TODO D-Bus server properties; address, port, interface, etc.           :2:
 
85
** Python 3                                                               :2:
 
86
*** TODO [#C] In Python 3.3, use shlex.quote() instead of re.escape()
92
87
 
93
88
* mandos-ctl
94
 
*** Handle "no D-Bus server" and/or "no Mandos server found" better
95
 
*** [#B] --dump option
96
 
** TODO Remove old string_to_delta format                                    :2:
 
89
** TODO Remove old string_to_delta format                                 :2:
97
90
 
98
91
* TODO mandos-dispatch
99
92
  Listens for specified D-Bus signals and spawns shell commands with
100
93
  arguments.
101
94
 
102
95
* mandos-monitor
 
96
** TODO --servicename                                                  :BUGS:
103
97
** TODO help should be toggleable
104
98
** Urwid client data displayer
105
99
   Better view of client data in the listing
106
100
*** Properties popup
107
101
** Print a nice "We are sorry" message, save stack trace to log.
108
 
** Rename module "gobject" to "GObject".
109
102
 
110
103
* mandos-keygen
111
104
** TODO "--secfile" option
118
111
*** TODO [#C] use same file name rules as run-parts(8)
119
112
*** TODO [#C] Do not install in initrd.img if configured not to.
120
113
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
121
 
** TODO [#C] /etc/bash_completion.d/mandos
 
114
** TODO [#C] $(pkg-config --variable=completionsdir bash-completion)
122
115
   From XML sources directly?
123
116
 
124
117
* Side Stuff
125
118
** TODO Locate which package moves the other bin/sh when busybox is deactivated
126
119
** TODO contact owner of package, and ask them to have that shell static in position regardless of busybox
127
120
 
 
121
* [[http://www.undeadly.org/cgi?action=article&sid=20110530221728][OpenBSD]]
 
122
 
128
123
 
129
124
#+STARTUP: showall