Univention Bugzilla – Full Text Bug Listing |
Summary: | S4C memory leak in group member cache | ||
---|---|---|---|
Product: | UCS | Reporter: | Arvid Requate <requate> |
Component: | S4 Connector | Assignee: | Arvid Requate <requate> |
Status: | CLOSED FIXED | QA Contact: | Florian Best <best> |
Severity: | normal | ||
Priority: | P5 | CC: | best |
Version: | UCS 4.4 | Flags: | requate:
Patch_Available+
|
Target Milestone: | UCS 4.4-3-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
See Also: | https://forge.univention.org/bugzilla/show_bug.cgi?id=50643 | ||
What kind of report is it?: | Development Internal | What type of bug is this?: | --- |
Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
User Pain: | Enterprise Customer affected?: | ||
School Customer affected?: | ISV affected?: | ||
Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
Ticket number: | Bug group (optional): | ||
Max CVSS v3 score: | |||
Attachments: |
fix_s4c_memory_leak_in_group_member_cache.patch
patch (git:fbest/50644-memory-leak) |
Created attachment 10274 [details]
patch (git:fbest/50644-memory-leak)
Alternative patch using python's set() which reduces the diff and simplifies the code.
e31e8d76ef | Turn self.group_members_cache_* into a set() 72479b1800 | Fix another self.group_members_cache_ucs type initialization 12d285dfd7 | Advisory OK: fix OK: Code review OK: jenkins-Tests OK: YAML |
Created attachment 10273 [details] fix_s4c_memory_leak_in_group_member_cache.patch From reading the S4-Connector code I think the s4().group_members_cache_* could be growing indefinitely, see attached patch.