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

Don't use g_error. Use g_warning instead and let the scanner to continue #710

Merged
merged 1 commit into from
Apr 27, 2021

Conversation

jjnicola
Copy link
Member

What:
Don't use g_error. Use g_warning instead and let the scanner to continue

Why:
Error messages are always fatal, resulting in a call to G_BREAKPOINT() to terminate the application. This function will result in a core dump; don't use it for errors you expect.

For more information see:
https://developer.gnome.org/glib/stable/glib-Message-Logging.html#g-error

How:
During a running scan, run a program which consume the whole free memory. The scanner will not fork() because no memory available.

Checklist:

…nue.

Error messages are always fatal, resulting in a call to G_BREAKPOINT() to terminate the application. This function will result in a core dump; don't use it for errors you expect.

For more information see:
https://developer.gnome.org/glib/stable/glib-Message-Logging.html#g-error
@ArnoStiefvater ArnoStiefvater merged commit eac4fd9 into greenbone:openvas-20.08 Apr 27, 2021
@jjnicola jjnicola deleted the error branch April 27, 2021 07:24
jjnicola added a commit that referenced this pull request Apr 27, 2021
Don't use g_error. Use g_warning instead and let the scanner to continue (backport #710)
jjnicola added a commit that referenced this pull request Apr 27, 2021
Don't use g_error. Use g_warning instead and let the scanner to continue (backport #710)
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.

2 participants