[tor-bugs] #20629 [Core Tor/Tor]: hs: Fix issues found by coverity

[Tor Bug Tracker & Wiki]

#20629: hs: Fix issues found by coverity
-----------------------------+------------------------------------
 Reporter:  dgoulet          |          Owner:  dgoulet
     Type:  defect           |         Status:  merge_ready
 Priority:  Medium           |      Milestone:  Tor: 0.3.0.x-final
Component:  Core Tor/Tor     |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:  tor-hs, prop224  |  Actual Points:
Parent ID:                   |         Points:  0.1
 Reviewer:                   |        Sponsor:  SponsorR-must
-----------------------------+------------------------------------

Comment (by nickm):

 > Is this about the fact that if tt_str_op() is triggered, we go to the
 done: label and then it leaks? I would be surprised as we often do that in
 test that is we assert and then we free.

 Actually, we have to fix that ''all the time'' :)

 Coverity will complain if there is any path where the memory is allocated
 but not freed.

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