linux-system-roles / storage

Ansible role for linux storage management
https://linux-system-roles.github.io/storage/
MIT License
101 stars 59 forks source link

Fix issues found by CodeQL #330

Closed nhosoi closed 1 year ago

nhosoi commented 1 year ago

library/resolve_blockdev.py

Additional fix based on the comment by @richm: https://github.com/linux-system-roles/storage/pull/329#issuecomment-1409087501

Thanks!

codecov-commenter commented 1 year ago

Codecov Report

Base: 16.54% // Head: 16.54% // No change to project coverage :thumbsup:

Coverage data is based on head (4e05d32) compared to base (52efa1b). Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #330 +/- ## ======================================= Coverage 16.54% 16.54% ======================================= Files 2 2 Lines 284 284 Branches 71 71 ======================================= Hits 47 47 Misses 237 237 ``` | Flag | Coverage Δ | | |---|---|---| | sanity | `16.54% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#carryforward-flags-in-the-pull-request-comment) to find out more. Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.