Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Trac Trac
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Issues 246
    • Issues 246
    • List
    • Boards
    • Service Desk
    • Milestones
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
  • Wiki
    • Wiki
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar
  • Legacy
  • TracTrac
  • Issues
  • #32376

Closed (moved)
(moved)
Open
Created Nov 05, 2019 by David Goulet@dgoulet🆘

test: Possible NULL deref in free_fake_orcirc()

Introduced with #32196 (moved).

Found by coverity:

*** CID 1455207:  Null pointer dereferences  (FORWARD_NULL)
/src/test/test_relay.c: 116 in test_relay_close_circuit()
110       if (orcirc) {
111         circuitmux_detach_circuit(nchan->cmux, TO_CIRCUIT(orcirc));
112         circuitmux_detach_circuit(pchan->cmux, TO_CIRCUIT(orcirc));
113         cell_queue_clear(&orcirc->base_.n_chan_cells);
114         cell_queue_clear(&orcirc->p_chan_cells);
115       }
>>>     CID 1455207:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "orcirc" to "free_fake_orcirc", which dereferences it.
116       free_fake_orcirc(orcirc);
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking