/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:
3
3
* [#A] README file
4
4
 
5
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
9
** [#B] Add more comments to code
8
10
** [#B] Add more if(debug) calls
9
11
** [#B] Seperate more code to function for more readability
 
12
** [#B] Make free_plugin_list() function
10
13
** [#A] Man page: man8/plugin-runner.8mandos
11
14
*** DESCRIPTION
12
15
    Describe the plus sign syntax for passing options from crypttab
25
28
    Text needed
26
29
*** SEE ALSO
27
30
    Explaining test on what you can read
28
 
** Use asprintf instead of malloc and strcat?
29
31
** Support in configuration file for environment variables
30
32
** Keydir move: /etc/mandos -> /etc/keys/mandos
31
33
   Must create in preinst if not pre-depending on cryptsetup
32
34
 
33
35
* Password-request
34
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]]
35
41
** [#B] Temporarily lower kernel log level
36
42
   for less printouts during sucessfull boot.
37
43
*** DESCRIPTION