/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
 
** TODO [#B] Prefer /run/tmp over /tmp, if it exists
26
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.
27
18
 
28
19
* splashy
29
20
** TODO [#B] use scandir(3) instead of readdir(3)
 
21
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
30
22
 
31
23
* usplash (Deprecated)
32
 
** TODO [#A] Make it work again
 
24
** TODO [#B] Make it work again
33
25
** TODO [#B] use scandir(3) instead of readdir(3)
 
26
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
34
27
 
35
28
* askpass-fifo
36
 
** TODO [#B] Drop privileges after opening FIFO.
 
29
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
37
30
 
38
31
* password-prompt
39
32
** TODO [#B] lock stdin (with flock()?)
 
33
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
40
34
 
41
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.
42
38
 
43
39
* TODO [#B] passdev
44
40
 
45
41
* plugin-runner
46
42
** TODO handle printing for errors for plugins
47
43
*** Hook up stderr of plugins, buffer them, and prepend "Mandos Plugin [plugin name]"
48
 
** TODO [#B] use scandir(3) instead of readdir(3)
49
44
** TODO [#C] use same file name rules as run-parts(8)
50
45
** kernel command line option for debug info
51
 
** TODO [#B] Use openat()
52
 
** TODO [#C] Use fnmatch() to look for bad prefixes and suffixes
 
46
** TODO [#A] Restart plugins which exit with EX_TEMPFAIL
53
47
 
54
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()
55
54
** TODO [#B] Log level                                                    :BUGS:
56
55
*** TODO /etc/mandos/clients.d/*.conf
57
56
    Watch this directory and add/remove/update clients?
66
65
    + Approve(False) -> Close client connection immediately
67
66
** TODO [#C] python-parsedatetime
68
67
** TODO Separate logging logic to own object
69
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
 
68
** TODO [#B] Limit approval_delay to max gnutls/tls timeout value
70
69
** TODO [#B] break the wait on approval_delay if connection dies
71
70
** TODO Generate Client.runtime_expansions from client options + extra
72
71
** TODO Allow %%(checker)s as a runtime expansion
73
 
** TODO Use python-tlslite?
74
72
** TODO D-Bus AddClient() method on server object
75
73
** TODO Use org.freedesktop.DBus.Method.NoReply annotation on async methods. :2:
76
 
** TODO Emit [[http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-properties][org.freedesktop.DBus.Properties.PropertiesChanged]] signal         :2:
77
 
   TODO Deprecate se.recompile.Mandos.Client.PropertyChanged - annotate!
78
 
   TODO Can use "invalidates" annotation to also emit on changed secret.
79
 
** TODO Support [[http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-objectmanager][org.freedesktop.DBus.ObjectManager]] interface on server object :2:
80
 
   Deprecate methods GetAllClients(), GetAllClientsWithProperties()
81
 
   and signals ClientAdded and ClientRemoved.
82
74
** TODO Save state periodically to recover better from hard shutdowns
83
75
** TODO CheckerCompleted method, deprecate CheckedOK
84
76
** TODO Secret Service API?
85
 
   http://standards.freedesktop.org/secret-service/
 
77
   https://standards.freedesktop.org/secret-service/
86
78
** TODO Remove D-Bus interfaces with old domain name                      :2:
87
79
** TODO Remove old string_to_delta format                                 :2:
88
 
** TODO --no-zeroconf (only valid if port or socket is set)
89
 
 
90
 
* mandos.xml
91
 
** Add mandos contact info in manual pages
 
80
** TODO http://0pointer.de/blog/projects/stateless.html
 
81
*** File in /usr/lib/sysusers.d to create user+group "_mandos"
 
82
** TODO Error handling on error parsing config files
 
83
** TODO init.d script error handling
 
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
89
** TODO Remove old string_to_delta format                                 :2:
97
90
 
98
91
* TODO mandos-dispatch
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