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

terminal: return NULL on error in terminal_signal_init #3174

Merged
merged 1 commit into from Oct 29, 2019

Conversation

@Blub
Copy link
Member

Blub commented Oct 29, 2019

Callers expect a NULL on error, and with PR #3171 marking
the pointer as __do_free, we now return a pointer to freed
memory here otherwise.

Signed-off-by: Wolfgang Bumiller w.bumiller@proxmox.com

Callers expect a NULL on error, and with PR #3171 marking
the pointer as __do_free, we now return a pointer to freed
memory here otherwise.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
@lxc-jenkins

This comment has been minimized.

Copy link

lxc-jenkins commented Oct 29, 2019

Testsuite passed

@stgraber stgraber merged commit 202d2ca into lxc:master Oct 29, 2019
4 checks passed
4 checks passed
Branch target Branch target is correct
Details
DCO All commits signed-off
Details
Testsuite Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.