Ticket 5917 - pmix mem leak patch for review and commit
Summary: pmix mem leak patch for review and commit
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: slurmstepd (show other tickets)
Version: 18.08.2
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Danny Auble
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2018-10-24 18:08 MDT by Moe Jette
Modified: 2018-11-05 10:39 MST (History)
1 user (show)

See Also:
Site: SchedMD
Slinky Site: ---
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
Google sites: ---
HPCnow Sites: ---
HPE Sites: ---
IBM Sites: ---
NOAA SIte: ---
NoveTech Sites: ---
Nvidia HWinf-CS Sites: ---
OCF Sites: ---
Recursion Pharma Sites: ---
SFW Sites: ---
SNIC sites: ---
Tzag Elita Sites: ---
Linux Distro: ---
Machine Name:
CLE Version:
Version Fixed: 18.08.4 19.05.0pre2
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
memory leak fix (433 bytes, patch)
2018-10-24 18:08 MDT, Moe Jette 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this ticket.
Description Moe Jette 2018-10-24 18:08:06 MDT 
Created attachment 8096 [details]
memory leak fix

Please mark the author of this patch as:
Artem Polyakov <artpol84@gmail.com>

The leak was introduced in this commit:
https://github.com/SchedMD/slurm/commit/bd827464d72204c6ef3e57e7740de6875bec07e1#diff-4ba0ea45b195656d1b85df5dc814cf4dL177
Comment 1 Artem Polyakov 2018-10-24 18:11:32 MDT 
Thanks, Moe!
Comment 2 Danny Auble 2018-11-05 10:39:41 MST 
Thanks guys, this is committed in e0000478eed3e8f.