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
prevent recursive logging csharp stackoverflow. #59503
prevent recursive logging csharp stackoverflow. #59503
Conversation
Bump? This is a simple fix, would be great to have it in please! @jborean93 |
Are you able to create a changelog fragment like https://github.com/ansible/ansible/blob/devel/changelogs/fragments/51995-apt_repository-catch-IOError.yaml. We require a changelog fragment to be able to backport modules to older branches. |
Yup, hopefully that suits. Do you need a rebase & squash or good as is? Cheers for the quick response :) |
bump @nitzmahone, have done the change log fragment too |
Sorry for the CI struggles there, looks good. Are you able to backport the PR to stable-2.8 so it's included in the next 2.8 release https://docs.ansible.com/ansible/latest/community/development_process.html#backporting-merged-prs. |
Thanks @jborean93 ! Done: #60267 |
* prevent recursive logging situation. * Add changelog (cherry picked from commit 6a5cd9a)
SUMMARY
Simple bugfix to avoid recursive function calls in C# code causing Stack overflow on windows / powershell. Note the already present [comment].(
ansible/lib/ansible/module_utils/csharp/Ansible.Basic.cs
Line 325 in 5228133
ISSUE TYPE
COMPONENT NAME
ansible.basics (core C#)
ADDITIONAL INFORMATION
To reproduce:
Call
win_ping
viavmware_tools
connection plugin. It will crash powershell and hang (whilst WerFault.exe dumps proc).vmware_tools
does not support privilege escalation and so the line:EventLog.CreateEventSource(logSource, "Application");
fails and we get into a cycle.