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

Re: [tor-bugs] #12951 [Tor]: BridgeAuth should add a `published` line to bridge networkstatus documents



#12951: BridgeAuth should add a `published` line to bridge networkstatus documents
------------------------+------------------------------------------------
     Reporter:  isis    |      Owner:
         Type:  defect  |     Status:  needs_review
     Priority:  minor   |  Milestone:
    Component:  Tor     |    Version:
   Resolution:          |   Keywords:  bridgedb-parsers, stem, bridgeauth
Actual Points:          |  Parent ID:
       Points:          |
------------------------+------------------------------------------------

Comment (by isis):

 As per Yawning's code review, I've removed the `tor_free(published);`
 line, due Yawning teaching me that statically-sized stack-based arrays
 don't need to be freed like dynamically-sized malloc'ed ones.

 The
 [https://gitweb.torproject.org/user/isis/tor.git/commitdiff/374b531dba966780f2e59163ca80b1b5a0b8f14c
 new, rebased changes] are in my `bug12951_r1`
 [https://gitweb.torproject.org/user/isis/tor.git/shortlog/refs/heads/bug12951_r1
 branch].

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/12951#comment:2>
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