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

Re: [tor-bugs] #18241 [Tor]: Assert that the event_base is initialized before using it.



#18241: Assert that the event_base is initialized before using it.
-------------------------------+------------------------------
 Reporter:  nickm              |          Owner:  nickm
     Type:  enhancement        |         Status:  needs_review
 Priority:  Medium             |      Milestone:
Component:  Tor                |        Version:
 Severity:  Normal             |     Resolution:
 Keywords:  TorCoreTeam201602  |  Actual Points:
Parent ID:                     |         Points:
  Sponsor:                     |
-------------------------------+------------------------------

Comment (by teor):

 It turns out that the log mutex isn't initialised the first time logv() is
 called.
 This causes logv to fail a tor_assert in LOCK_LOGS(), which calls
 logv()...

 Please see my branch check_log_mutex for:
 * the check that exposes this issue,
 * further checks that ensure we don't log when SMARTLIST_DEBUG is defined,
 * a change to the initialiser order so we init logs first (should we do
 this in tor, too?)

 I needed to cherry-pick 1bac468882fd732460d8a25735131d632f977bfe to get
 this branch to work.

 tor_mutex_init() can still call tor_assert if it fails, so the possibility
 of this stack overflow still exists when we're initialising the log mutex
 itself.
 What do you want to do about that one?

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