external_libraries/oscpack/TODO

   1 TODO:
   2
   3     - consider adding the local endpoint name to PacketListener::PacketReceived() params
   4
   5     - consider adding ListenerThread class to support old seperate thread listener functionality, something like:
   6
   7         class UdpSocketListenerThread{
   8         public:
   9             UdpSocketListenerThread( UdpSocket& socket, Listener *listener );
  10             UdpSocketListenerThread( UdpSocketReceiveMultiplexer *mux );
  11             ~UdpSocketListenerThread();
  12
  13             void Run();
  14             void Stop();
  15         };
  16
  17     - provide some kind of automatic endianness configuration (hopefully there
  18         are gcc symbols for this)
  19
  20     - work out a way to make the parsing classes totally safe. at a minimum this
  21     means adding functions to test for invalid float/doublevalues,
  22     making sure the iterators never pass the end of the message, ...
  23         (passing end of message can happen if:
  24             - too many args in type tags
  25                 a. typetags overflow message size
  26                 b. args fulfilling typetags overflow message size
  27             - strings too long or not terminated correctly
  28             - blobs too long or not terminated correctly
  29
  30         if the message was fully checked during construction, the end() iterator
  31         could be moved back until only arguments which fit withing size() may
  32         be interated (this could be none). A flag could be set to indicate that
  33         something was wrong.
  34
  35     - other packet badness could include:
  36         - time tags too far into the future (the scheduler should deal with
  37             that i guess).
  38         - message address patterns which aren't correctly terminated
  39
  40     - improve the ability to parse messages without tags (SC uses methods which
  41             get the data and advance the iterator in one step.)
  42         - Check* could be modified to do this - ie if typetags are not present
  43             it could check that reading the field won't escape the message size
  44             and return the data, or return false if some consistency
  45             constraint is violated.
  46         (or alternately drop support for messages without type tags)
  47
  48
  49     - add a method to discard an inprogress message if it gets half
  50         constructed and the buffer is full in OutboundPacket
  51
  52     - write a stress testing app which can send garbage packets to try to flush out other bugs in the parsing code.
  53
  54
  55