#2869 cache_req tests don't use leak_check_push/leak_check_pop in fixtures
Closed: Fixed None Opened 4 years ago by jhrozek.

During review of patches for #2730, we noticed the tests should use check_leaks_push/pop in the test fixtures to catch issues where memory was allocated on long-living contexts (like responder or event context).

Triage notes - this should be a prerequisite for using cache_req in responders (#1126) which is currently 1.14.

For the time being, filing into 1.14 beta, we will decide if there is still time to use cache_req in 1.14 or whether we need to move that enhancement to 1.15..

milestone: NEEDS_TRIAGE => SSSD 1.14 beta

Fields changed

rhbz: => 0

Pavel agreed on IRC to fix this ticket.

owner: somebody => pbrezina

Fields changed

status: new => assigned

Fields changed

patch: 0 => 1

Fields changed

milestone: SSSD 1.14 beta => SSSD 1.14 alpha
resolution: => fixed
status: assigned => closed

Metadata Update from @jhrozek:
- Issue assigned to pbrezina
- Issue marked as blocked by: #1126
- Issue set to the milestone: SSSD 1.14 alpha

2 years ago

Login to comment on this ticket.