[tor-bugs] #13460 [Tor Browser]: fix error handling mistake in patch for Bug 13028

Tor Bug Tracker & Wiki blackhole at torproject.org
Fri Oct 24 05:25:55 UTC 2014


#13460: fix error handling mistake in patch for Bug 13028
-----------------------------+---------------------------------------------
     Reporter:               |      Owner:
  arthuredelstein            |     Status:  closed
         Type:  defect       |  Milestone:
     Priority:  normal       |    Version:
    Component:  Tor Browser  |   Keywords:  MikePerry201410R, tbb-4.5-alpha
   Resolution:  fixed        |  Parent ID:
Actual Points:               |
       Points:               |
-----------------------------+---------------------------------------------

Comment (by arthuredelstein):

 Replying to [comment:3 mikeperry]:
 > Couple points:
 >
 > 1. In one of those two cases, I used the return because the normal error
 handling had a free for a strdup. We need to move the block above the
 strdup if we want to use it.
 > 2. Your fixup commits need to have the full commit summary line that
 they are fixing, otherwise git won't autosquash them together when we
 rebase.

 Both good points -- thanks.

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


More information about the tor-bugs mailing list