[tor-bugs] #15826 [Pluggable transport]: Check and return error values in goptlib

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Jun 24 19:22:36 UTC 2015


#15826: Check and return error values in goptlib
-------------------------------------+----------------------------
     Reporter:  gsathya              |      Owner:  asn
         Type:  defect               |     Status:  needs_revision
     Priority:  normal               |  Milestone:
    Component:  Pluggable transport  |    Version:
   Resolution:                       |   Keywords:  goptlib
Actual Points:                       |  Parent ID:
       Points:                       |
-------------------------------------+----------------------------

Comment (by dcf):

 Replying to [comment:1 dcf]:
 > > git.torproject.org/pluggable-transports/goptlib.git/pt.go:857:15
 s.SetDeadline(time.Now().Add(5 * time.Second))
 > > git.torproject.org/pluggable-transports/goptlib.git/pt.go:868:15
 s.SetDeadline(time.Time{})
 >
 > These ones look good.

 Now that I look again, these need to `s.Close()` in the error case.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/15826#comment:2>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list