added notes about what to fix next

diff --git a/todo/TODO b/todo/TODO
index f16d8a8..4a97631 100644 (file)
--- a/todo/TODO
+++ b/todo/TODO
@@ -1,5 +1,9 @@
 URGENT:
 URGENT:

-o Add 'initialize' list in modules, so explicitely give initialization order.
+o scan ALL modules, so that they use consistent names for structures
+  and variables in their instantiation (rlm_foo_t *inst), and that they
+  call rlm_foo_detach() if anything goes wrong, instead of just free()'ing
+  'inst'.  The cf_parse_section().. code may have malloc'd memory, and
+  that needs to be free'd, too.

 o Add a 'listen' directive, ala Apache, so that we can listen
   on multiple ports. Hmm... if we do that, we should allow multiple
   virtual servers, too.
 o Add a 'listen' directive, ala Apache, so that we can listen
   on multiple ports. Hmm... if we do that, we should allow multiple
   virtual servers, too.


@@ -11,7 +15,6 @@ o double-check Cistron 'compat' mode, so that all current users of
   Cistron can upgrade "out of the box" to FreeRADIUS
 
 MEDIUM PRIORITY
   Cistron can upgrade "out of the box" to FreeRADIUS
 
 MEDIUM PRIORITY

-o fix radwho to read modules{radutmp{filename = }}

 o go through all of the code and reformat it, for project standards
 o go through all of the code, removing unnecessary #include's, and
   generating a standard include file order which will work across all
 o go through all of the code and reformat it, for project standards
 o go through all of the code, removing unnecessary #include's, and
   generating a standard include file order which will work across all


@@ -63,6 +66,8 @@ o there should be a way that radius itself could
        --- This work is for an external process to do ---
 
 DONE:
        --- This work is for an external process to do ---
 
 DONE:

+o fix radwho to read modules{radutmp{filename = }}
+o Add 'initialize' list in modules, so explicitely give initialization order.

 o merge OSF/OSFIA patches from Cistron.
 o Fix DBM support:
   - Put DBM into its own module
 o merge OSF/OSFIA patches from Cistron.
 o Fix DBM support:
   - Put DBM into its own module