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

Failures in get_peak_rss() should fail the application instead of silently returning 0 #763

Open
1 task done
yurivict opened this issue Aug 31, 2022 · 0 comments
Open
1 task done

Comments

@yurivict
Copy link
Contributor

Prerequisites

  • Put an X between the brackets on this line if you have done all of the following:
    • Checked that your issue isn't already filed.
    • Reduced the issue to a self-contained, reproducible test case.

Description

get_peak_rss() on FreeBSD was always failing due to a wrong /proc/... path and all tests still succeeded.

This function should be changed to hard-fail when any system call fails. In its current condition it might be silently failing for whatever reasons and users are never notified.

@yurivict yurivict changed the title Failures in get_peak_rss() shoiuld fail application instead of silently returing 0 Failures in get_peak_rss() should fail application instead of silently returning 0 Aug 31, 2022
@yurivict yurivict changed the title Failures in get_peak_rss() should fail application instead of silently returning 0 Failures in get_peak_rss() should fail the application instead of silently returning 0 Aug 31, 2022
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

No branches or pull requests

1 participant