/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: 2008-08-16 03:29:08 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080816032908-ihw7c05r2mnyk389
Add feature to specify custom environment variables for plugins.

* plugin-runner.c (plugin): New members "environ" and "envc" to
                            contain possible custom environment.
  (getplugin): Return NULL on failure instead of doing exit(); all
               callers changed.
  (add_to_char_array): New helper function for "add_argument" and
                       "add_environment".
  (addargument): Renamed to "add_argument".  Return bool.  Call
                 "add_to_char_array" to actually do things.
  (add_environment): New; analogous to "add_argument".
  (addcustomargument): Renamed to "add_to_argv" to avoid confusion
                       with "add_argument".
  (main): New options "--global-envs" and "--envs-for" to specify
          custom environment for plugins.  Print environment for
          plugins in debug mode.  Use asprintf instead of strcpy and
          strcat.  Use execve() for plugins with custom environments.
          Free environment for plugin when freeing plugin list.

Show diffs side-by-side

added added

removed removed

Lines of Context:
TODO
2
2
 
3
3
* [#A] README file
4
4
 
5
 
* Mandos-client
 
5
* Plugin-runner
 
6
** [#A] Free plugin name and args
 
7
   [[file:plugin-runner.c::free%20plugin_list][file:plugin-runner.c::free plugin_list]] (both places)
6
8
** [#A] Change syntax for arguments
7
 
** [#A] Man page: man8/mandos-client.8mandos
 
9
** [#B] Add more comments to code
 
10
** [#B] Add more if(debug) calls
 
11
** [#B] Seperate more code to function for more readability
 
12
** [#B] Make free_plugin_list() function
 
13
** [#A] Man page: man8/plugin-runner.8mandos
8
14
*** DESCRIPTION
9
15
    Describe the plus sign syntax for passing options from crypttab
10
16
*** EXIT STATUS
22
28
    Text needed
23
29
*** SEE ALSO
24
30
    Explaining test on what you can read
25
 
** Use asprintf instead of malloc and strcat?
26
 
** use strsep instead of strtok?
27
 
** use config file in addition to arguments
28
 
** pass things in environment, like device name, etc
 
31
** Support in configuration file for environment variables
 
32
** Keydir move: /etc/mandos -> /etc/keys/mandos
 
33
   Must create in preinst if not pre-depending on cryptsetup
29
34
 
30
35
* Password-request
31
36
** [#A] Man page: man8/password-request.8mandos
 
37
** Make prompt exactly like the normal prompt
 
38
   Use environment variables:
 
39
   "Enter passphrase to unlock the disk $cryptsource ($crypttarget): "
 
40
   [[file:plugins.d/password-prompt.c::fprintf%20stderr%20s%20Password%20prefix][Here]]
32
41
** [#B] Temporarily lower kernel log level
33
42
   for less printouts during sucessfull boot.
34
43
*** DESCRIPTION
54
63
    Create this section
55
64
*** SEE ALSO
56
65
    Refer to mandos-client(8mandos) and password-prompt(8mandos)
57
 
** Use asprintf instead of malloc and memcpy?
58
66
** IPv4 support
59
67
** use strsep instead of strtok?
60
68
** Do not depend on GnuPG key rings on disk
63
71
 
64
72
* Password-prompt
65
73
** [#A] Man page: man8/password-prompt.8mandos
 
74
** Show getenv("cryptsource") and getenv("crypttarget") if set
 
75
   cryptsource will be the device, like "/dev/sda3", and crypttarget
 
76
   will be the device mapper name, like "sda3_crypt".
66
77
*** DESCRIPTION
67
78
    Move options to new OPTIONS section.
68
79
*** EXIT STATUS