TODO/bugs In SILC Server
========================
+ o The SKE protocol in the server does not verify the remote hosts
+ (the router's) public key at all. All public keys are accepted without
+ verification - this obviously is not secure.
+
o When server quits and all clients of that server are removed from all
channels the channel keys are re-generated for all clients. This is
a bug and should be done only once per channel after all clients of
own resolver stuff (through scheduler, if possible without writing
too much own stuff) or use threads.
- o The ID List must be optimized. When the lists grow the searching
- becomes a lot slower and is some cases the lists are searched many
- times, like with channel messages (twice at least). Some sort of
- hash tables should replace the lists. Thus, the ID cache should be
- rewritten to use hash tables internally.
-
o The backup router support described in the protocol specification
should be done at some point.
o All network routines in lib/silcutil/silcnet.[ch] does not
support IPv6.
-
- o Hash tables must be implemented. The requirement for this is that
- the hash table is collision resistant so that it can be used in
- critical positions as well. It probably works the 95% of the time
- fine without collisions but the last 5% of the cases must be
- handled. Maybe two interfaces could be done, one for normal static
- hash tables and one for collision resistant hash table.
+ o silc_id_render supports only IPv4 based ID's in the file
+ lib/silcutil/silcutil.c.
o Compression routines are missing. The protocol supports packet
compression thus it must be implemented. SILC Comp API must be
o The CAST cipher is not compiled currently due to compilation errors;
check those. Cast is in lib/silccrypt/cast.c.
- o SIMs did not compile under cygwin.
+ o All payload parsing (decoding) functions should take unsigned char *
+ and uint32 as data and data length as arguments. Now some of the
+ routines do already that but most of the routines use SilcBuffer.
+ The SilcBuffer ones should be removed since buf->data and buf->len
+ is more convenient to use. However, the silc_buffer_[un]format
+ routines support only SilcBuffer so they would require reallocation
+ of SilcBuffer. Maybe support for raw data (and not just SilcBuffer)
+ should be added silc_buffer_[un]format_? routines. These are currently
+ only cosmetic changes but at some point must be done to make the
+ payload interfaces consistent.
TODO After 1.0