Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory leak in the simple access provider #5456

Closed
sumit-bose opened this issue Jan 12, 2021 · 2 comments
Closed

Memory leak in the simple access provider #5456

sumit-bose opened this issue Jan 12, 2021 · 2 comments
Assignees
Labels
Bugzilla Closed: Fixed Issue was closed as fixed.

Comments

@sumit-bose
Copy link
Contributor

Ticket was cloned from Red Hat Bugzilla (product Red Hat Enterprise Linux 8): Bug 1915395

Description of problem:
The simple access provider has a memory leak while re-reading the access and
deny lists.
sumit-bose added a commit to sumit-bose/sssd that referenced this issue Jan 12, 2021
The simple access provider with reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider
sumit-bose added a commit to sumit-bose/sssd that referenced this issue Jan 14, 2021
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider
@pbrezina
Copy link
Member

Pushed PR: #5457

  • master
    • 19c2c64 - simple: fix memory leak while reloading lists

@pbrezina pbrezina added the Closed: Fixed Issue was closed as fixed. label Jan 15, 2021
elkoniu pushed a commit to elkoniu/sssd that referenced this issue Feb 28, 2021
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
akuster pushed a commit to akuster/sssd that referenced this issue May 18, 2021
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
sumit-bose added a commit to sumit-bose/sssd that referenced this issue Jul 31, 2021
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider

(cherry picked from commit d16e0b6)
sumit-bose added a commit to sumit-bose/sssd that referenced this issue Aug 5, 2021
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
(cherry picked from commit 19c2c64)
pbrezina pushed a commit that referenced this issue Aug 9, 2021
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: #5456

:fixes: Memory leak in the simple access provider

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
(cherry picked from commit 19c2c64)

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
@pbrezina
Copy link
Member

pbrezina commented Aug 9, 2021

Pushed PR: #5731

  • sssd-1-16
    • 137924c - simple: fix memory leak while reloading lists

etrunko pushed a commit to etrunko/sssd that referenced this issue Nov 16, 2023
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
(cherry picked from commit 19c2c64)

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
etrunko pushed a commit to etrunko/sssd that referenced this issue Nov 16, 2023
The simple access provider will reload the access and deny lists at
runtime to make sure that users and groups from domains which are
discovered at runtime are properly processed.

While reloading the lists the original lists are not freed and an
intermediate list wasn't removed as well.

Resolves: SSSD#5456

:fixes: Memory leak in the simple access provider

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
(cherry picked from commit 19c2c64)

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugzilla Closed: Fixed Issue was closed as fixed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants