[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #1922 [Tor]: torrc.d-style configuration directories



#1922: torrc.d-style configuration directories
-----------------------------+------------------------------
     Reporter:  aa138346     |      Owner:
         Type:  enhancement  |     Status:  needs_revision
     Priority:  normal       |  Milestone:  Tor: unspecified
    Component:  Tor          |    Version:
   Resolution:               |   Keywords:  tor-client
Actual Points:               |  Parent ID:
       Points:               |
-----------------------------+------------------------------
Changes (by nickm):

 * status:  needs_review => needs_revision


Comment:

 This needs more work than I can give it, but I should say some stuff:
   * It would be a good idea to learn how to do safe string handling in C.
 strcat is just as bad as strcpy.  I do not want heap overflows in tor.
   * "/etc/tor.d" is still hardwired.
   * It still uses a fake command line.
   * It fails badly if smartlist_len(dirlist) is greater than CHAR_MAX.
 Why would you ever use a char for that?
   * concatenating files is probably not the right approach.

 These are just the basic issues; there are lots of comments and design
 discussions I made above that are still unfixed.  I think this whole thing
 needs a total rewrite.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/1922#comment:22>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs