Updated automake.
[libgcrypt.git] / TODO
1 What's left to do                                 -*- outline -*-
2
3 * udiv-qrnbd.o should get build as *.lo [HPUX]
4
5 * Allow operation using RSA keys consisting of the OpenSSL list of
6   parameters and allow for a third form where the private Exponent
7   is not given (saves space).
8
9 * Add a warning to the manual, to check that libgcrypt actually has
10   been compiled with thread support when used by a threaded
11   application.
12
13 * linker script test
14   Write an autoconf test to check whether the linker supports a
15   version script. 
16
17 * Make use of the forthcoming visibility attribute.
18
19 * Add attributes to the MPI functions.
20
21 * In case the ac interface will be more popular than the pk interface,
22   the asymmetric ciphers could be changed for convenient interaction
23   with the ac interface (i.e. by using ac's `data sets') and the pk
24   interface could be changed to be a wrapper for the ac interface.
25
26 * cipher/pubkey.c and pubkey implementaions.
27   Don't rely on the secure memory based wiping function but add an
28   extra wiping.
29   
30 * update/improve documentation
31 ** it's outdated for e.g. gcry_pk_algo_info.
32 ** document algorithm capabilities
33 ** Explain seed files and correlation
34   Multiple instances of the applications sharing the same random seed
35   file can be started in parallel, in which case they will read out
36   the same pool and then race for updating it (the last update
37   overwrites earlier updates).  They will differentiate only by the
38   weak entropy that is added in read_seed_file based on the PID and
39   clock, and up to 16 bytes of weak random non-blockingly.  The
40   consequence is that the output of these different instances is
41   correlated to some extent.  In the perfect scenario, the attacker
42   can control (or at least guess) the PID and clock of the
43   application, and drain the system's entropy pool to reduce the "up
44   to 16 bytes" above to 0.  Then the dependencies of the inital states
45   of the pools are completely known.
46 ** Init requirements for random
47    The documentation says in "Controlling the library" that some
48    functions can only be used at initialization time, but it does not
49    explain what that means.  Initialization is a multi-step procedure:
50    First the thread callbacks have to be set up (optional), then the
51    gcry_check_version() function must be called (mandatory), then
52    further functions can be used.
53
54    The manual also says that something happens when the seed file is
55    registered berfore the PRNG is initialized, but it does not say how
56    one can guarantee to call it early enough.
57
58    Suggested fix: Specify initialization time as the time after
59    gcry_check_version and before calling any other function except
60    gcry_control().
61
62    All functions which modify global state without a lock must be
63    documented as "can only be called during initialization time" (but
64    see item 1).  Then the extraneous calls to _gcry_random_initialize
65    in gcry_control() can be removed, and the comments "not thread
66    safe" in various initialization-time-only functions like
67    _gcry_use_random_daemon become superfluous.
68
69 * Use builtin bit functions of gcc 3.4
70
71 * Consider using a daemon to maintaint he random pool
72   [Partly done] The down side of this is that we can't assume that the
73   random has has always been stored in "secure memory".  And we rely
74   on that sniffing of Unix domain sockets is not possible.  We can
75   implement this simply by detecting a special prefixed random seed
76   name and divert in this case to the daemon.  There are several
77   benefits with such an approach: We keep the state of the RNG over
78   invocations of libgcrypt based applications, don't need time
79   consuming initialization of the pool and in case the entropy
80   collectros need to run that bunch of Unix utilities we don't waste
81   their precious results.
82
83 * Out of memory handler for secure memory should do proper logging
84
85   There is no shortage of standard memory, so logging is most likely
86   possible.
87
88 * mpi_print does not use secure memory
89   for internal variables.
90
91 * gcry_mpi_lshift is missing
92
93 * Add internal versions of mpi functions
94   Or make use of the visibility attribute.
95
96 * Add OAEP
97
98 * Next API break:
99 ** gcry_ac_io_t
100   Remove use of anonymous union.
101
102 * ac.c
103   There are still some things fishy.  The fixes I did today
104   (2006-10-23) seem to cure just a symptom.  Needs a complete review.
105
106 * gcryptrnd.c
107   Requires a test for pth [done] as well as some other tests.
108
109 * random.c
110  If add_randomness is invoked before the pool is filled, but with a
111  weak source of entropy, for example the fast random poll, which
112  may happen from other parts of gcrypt, then the pool is filled
113  partially with weak random, defeating the purpose of pool_filled
114  and the "source > 1" check in add_randomness.
115
116  Suggestion: Count initial filling bytes with source > 1 in
117  add_randomness seperately from the pool_writepos cursor.  Only set
118  pool_filled if really POOLSIZE bytes with source > 1 have been
119  added.
120
121 * secmem.c
122   Check whether the memory block is valid before releasing it and
123   print a diagnosic, like glibc does.
124
125 * threads
126 ** We need to document fork problems
127   In particular that reinitialization is required in random.c
128   However, there is no code yet to do it.
129
130 * Tests
131   We need a lot more tests.  Lets keep an ever growing list here.
132 ** Write tests for the progress function
133 ** mpitests does no real checks yet.
134 ** pthreads
135   To catch simple errors like the one fixed on 2007-03-16.
136 ** C++ tests
137   We have some code to allow using libgcrypt from C++, so we also
138   should have a test case.