SSSD / sssd

#879 sysdb_attrs_get_el() should take a mem_ctx for the returned element
Closed: Invalid Opened by sgallagh.

Closed: Invalid
Reopen Issue

Currently, this function (and the various functions that wrap it) attaches the memory for the returned {{{struct ldb_message_element}}} to the {{{attrs}} input argument. This is a bad idea, as it makes it very easy to end up with memory leaks.

This function and its wrappers should include a mem_ctx input argument to ensure that the proper memory hierarchy is maintained for the returned values.

This behavior is not actually incorrect (though it is ugly). This may be resolved when ticket #629 is fixed.

For now, closing as INVALID.

resolution: => invalid
status: new => closed

Fields changed

rhbz: => 0

Fields changed

milestone: NEEDS_TRIAGE => void

Metadata Update from @sgallagh:
- Issue set to the milestone: void

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/1921

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.
Metadata
None
None
None
None
Normal
task
SysDB
1.5.7
None
0
None
0
0
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.