{"action":"created","issue":{"url":"https://api.github.com/repos/PBSPro/pbspro/issues/832","repository_url":"https://api.github.com/repos/PBSPro/pbspro","labels_url":"https://api.github.com/repos/PBSPro/pbspro/issues/832/labels{/name}","comments_url":"https://api.github.com/repos/PBSPro/pbspro/issues/832/comments","events_url":"https://api.github.com/repos/PBSPro/pbspro/issues/832/events","html_url":"https://github.com/PBSPro/pbspro/pull/832","id":366422124,"node_id":"MDExOlB1bGxSZXF1ZXN0MjIwMTAwMDgz","number":832,"title":"Resolve req_resvSub() memory leak","user":{"login":"ashwathraop","id":17821341,"node_id":"MDQ6VXNlcjE3ODIxMzQx","avatar_url":"https://avatars3.githubusercontent.com/u/17821341?v=4","gravatar_id":"","url":"https://api.github.com/users/ashwathraop","html_url":"https://github.com/ashwathraop","followers_url":"https://api.github.com/users/ashwathraop/followers","following_url":"https://api.github.com/users/ashwathraop/following{/other_user}","gists_url":"https://api.github.com/users/ashwathraop/gists{/gist_id}","starred_url":"https://api.github.com/users/ashwathraop/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ashwathraop/subscriptions","organizations_url":"https://api.github.com/users/ashwathraop/orgs","repos_url":"https://api.github.com/users/ashwathraop/repos","events_url":"https://api.github.com/users/ashwathraop/events{/privacy}","received_events_url":"https://api.github.com/users/ashwathraop/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-10-03T16:25:01Z","updated_at":"2018-10-04T14:12:05Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/PBSPro/pbspro/pulls/832","html_url":"https://github.com/PBSPro/pbspro/pull/832","diff_url":"https://github.com/PBSPro/pbspro/pull/832.diff","patch_url":"https://github.com/PBSPro/pbspro/pull/832.patch"},"body":"<!--- Please review your changes in preview mode -->\r\n<!--- Provide a general summary of your changes in the Title above -->\r\n\r\n#### Bug/feature Description\r\n* Memory leak during reservation submission with wall time requests*\r\n\r\n#### Affected Platform(s)\r\n* All\r\n\r\n#### Cause / Analysis / Design\r\n* When a reservation is submitted with wall time requests PBS errors out without freeing the memory allocated to hold the reservation information\r\n\r\n#### Solution Description\r\n* Free the memory before we send req_reject()\r\n\r\n#### Testing logs/output\r\n[reservation.txt](https://github.com/PBSPro/pbspro/files/2442817/reservation.txt)\r\n[resv_start_end.txt](https://github.com/PBSPro/pbspro/files/2442818/resv_start_end.txt)\r\n[smoke.txt](https://github.com/PBSPro/pbspro/files/2442819/smoke.txt)\r\n\r\n\r\n#### Checklist:\r\n<!--- Use the preview button to see the checkboxes/links properly. -->\r\n<!--- Go over the following points, and put an `x` (without spaces around it) in the boxes that apply. -->\r\n<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->\r\n- [x] I have joined the **[pbspro community forum](http://community.pbspro.org/)**.\r\n- [x] My pull request contains a **single, signed** commit. See **[setting up gpg signature](https://pbspro.atlassian.net/wiki/display/DG/Signing+Your+Git+Commits)**.\r\n- [x] My code follows the **[coding style](https://pbspro.atlassian.net/wiki/display/DG/Coding+Standards)** of this project.\r\n- [ ] My change requires project documentation. See **[required documentation checklist](https://pbspro.atlassian.net/wiki/display/DG/Checklist+for+Developing+Features+and+Bug+Fixes)** for details.\r\n   - [ ] I have added documentation in the **[project documentation area](https://pbspro.atlassian.net/wiki/display/PD)**.\r\n- [ ] I have added new **PTL test(s) to my commit**. (See **[using PTL for testing](https://pbspro.atlassian.net/wiki/display/DG/Using+PTL+for+Testing)**) *(or)*\r\n   - [ ] I have added  **manual test(s) to this pull request and explained why PTL is not appropriate** for this case.\r\n- [x] All new and existing automated tests have passed. (See **[running automated PTL tests](https://pbspro.atlassian.net/wiki/display/DG/PTL+Quick+Start+Guide)**).\r\n- [x] I have attached **test logs to this pull request** as evidence of testing/verification.\r\n\r\n\r\n__***For further information please visit the [Developer Guide Home](https://pbspro.atlassian.net/wiki/display/DG/Developer+Guide+Home).***__\r\n"},"comment":{"url":"https://api.github.com/repos/PBSPro/pbspro/issues/comments/427034280","html_url":"https://github.com/PBSPro/pbspro/pull/832#issuecomment-427034280","issue_url":"https://api.github.com/repos/PBSPro/pbspro/issues/832","id":427034280,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNzAzNDI4MA==","user":{"login":"arungrover","id":17924134,"node_id":"MDQ6VXNlcjE3OTI0MTM0","avatar_url":"https://avatars0.githubusercontent.com/u/17924134?v=4","gravatar_id":"","url":"https://api.github.com/users/arungrover","html_url":"https://github.com/arungrover","followers_url":"https://api.github.com/users/arungrover/followers","following_url":"https://api.github.com/users/arungrover/following{/other_user}","gists_url":"https://api.github.com/users/arungrover/gists{/gist_id}","starred_url":"https://api.github.com/users/arungrover/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arungrover/subscriptions","organizations_url":"https://api.github.com/users/arungrover/orgs","repos_url":"https://api.github.com/users/arungrover/repos","events_url":"https://api.github.com/users/arungrover/events{/privacy}","received_events_url":"https://api.github.com/users/arungrover/received_events","type":"User","site_admin":false},"created_at":"2018-10-04T14:12:05Z","updated_at":"2018-10-04T14:12:05Z","author_association":"CONTRIBUTOR","body":"Thanks Ashwath for posting the logs.\r\nPlease the the branch."}}
{"actor":{"display_login":"arungrover"}}