[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