Re: [tor-bugs] #19733 [Applications/Tor Browser]: GETINFO response parser doesn't handle AF_UNIX entries.

["Tor Bug Tracker & Wiki" <blackhole@xxxxxxxxxxxxxx>]

#19733: GETINFO response parser doesn't handle AF_UNIX entries.
-------------------------------------------------+-------------------------
 Reporter:  yawning                              |          Owner:  tbb-
                                                 |  team
     Type:  defect                               |         Status:
                                                 |  needs_review
 Priority:  Very Low                             |      Milestone:
Component:  Applications/Tor Browser             |        Version:
 Severity:  Minor                                |     Resolution:
 Keywords:  tbb-sandbox, tbb-torbutton,          |  Actual Points:
  TorBrowserTeam201609R                          |
Parent ID:  #14270                               |         Points:
 Reviewer:                                       |        Sponsor:
-------------------------------------------------+-------------------------

Comment (by arthuredelstein):

 Replying to [comment:7 mcs]:
 > Here is a new patch:
 >
 https://gitweb.torproject.org/user/brade/torbutton.git/commit/?h=bug19733-02&id=f1932e73ee5199969f49ce595fb30ec14e76cb52
 Minor things:
 {{{
 + if (socksAddr && (socksAddr.substr(0, 5) == "file:")) {
 }}}
 Can this be `sockAddr.startsWith("file:")`?
 {{{
 + foundSocksListener = (socketPath == path);
 }}}
 {{{
 +          foundSocksListener = ((socksAddr == torSocksAddr) &&
 +                                (socksPort == torSocksPort));
 }}}
 I would suggest using `===` (triple equals) for safety.

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