Skip to content

[202311][TACACS] Fix memory leak when authenticating using tacacs#23226

Merged
yxieca merged 1 commit intosonic-net:202311from
DavidZagury:202311_tacacs_mem
Jul 24, 2025
Merged

[202311][TACACS] Fix memory leak when authenticating using tacacs#23226
yxieca merged 1 commit intosonic-net:202311from
DavidZagury:202311_tacacs_mem

Conversation

@DavidZagury
Copy link
Contributor

Cherry-pick of #22600

Why I did it

A small memory leak was found when running TACACS

Work item tracking
  • Microsoft ADO (number only):

How I did it

A patch was created for the libnss-tacplus repository that will fix the memory leak.

How to verify it

Configure TACACS
Use the python script that reproduce the consumption and monitor its memory consumption.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Why I did it
A small memory leak was found when running TACACS

How I did it
A patch was created for the libnss-tacplus repository that will fix the memory leak.

How to verify it
Configure TACACS
Use the python script that reproduce the consumption and monitor its memory consumption.
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DavidZagury
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@qiluo-msft qiluo-msft requested a review from yxieca July 7, 2025 16:48
@DavidZagury
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 8e95631 into sonic-net:202311 Jul 24, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants