[tor-bugs] #32356 [Core Tor/Tor]: hs-v3: Memory leak in rend_client_get_random_intro_impl()

[Tor Bug Tracker & Wiki]

#32356: hs-v3: Memory leak in rend_client_get_random_intro_impl()
--------------------------------+--------------------------------
     Reporter:  dgoulet         |      Owner:  (none)
         Type:  defect          |     Status:  new
     Priority:  Medium          |  Milestone:  Tor: 0.4.3.x-final
    Component:  Core Tor/Tor    |    Version:
     Severity:  Normal          |   Keywords:  tor-hs
Actual Points:                  |  Parent ID:
       Points:  0.1             |   Reviewer:
      Sponsor:  Sponsor27-must  |
--------------------------------+--------------------------------
 From coverity report:

 {{{
 ** CID 1455168:  Resource leaks  (RESOURCE_LEAK)
 /src/feature/rend/rendclient.c: 1061 in
 rend_client_get_random_intro_impl()


 ________________________________________________________________________________________________________
 *** CID 1455168:  Resource leaks  (RESOURCE_LEAK)
 /src/feature/rend/rendclient.c: 1061 in
 rend_client_get_random_intro_impl()
 1055       usable_nodes = smartlist_new();
 1056       smartlist_add_all(usable_nodes, entry->parsed->intro_nodes);
 1057
 1058       /* Get service ID so we can use it to query the failure cache.
 If we fail to
 1059        * parse it, this cache entry is no good. */
 1060       if (BUG(rend_get_service_id(entry->parsed->pk, service_id) <
 0)) {
 >>>     CID 1455168:  Resource leaks  (RESOURCE_LEAK)
 >>>     Variable "usable_nodes" going out of scope leaks the storage it
 points to.
 1061         return NULL;
 1062       }
 }}}

 Was just introduced couple days ago.

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