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

swtpm_setup: Refactor code related to reading of the config file #920

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

stefanberger
Copy link
Owner

No description provided.

…function

Move the checking of access to and reading of the configuration file into
its own function.

Signed-off-by: Stefan Berger <[email protected]>
Return an error if the reading of the config file failed so that
config_file_lines can never be NULL. Remove all checks for
config_file_lines == NULL.

It's very unlikely reading of the config file failed since there's a file
access check right before it.

Signed-off-by: Stefan Berger <[email protected]>
@stefanberger stefanberger force-pushed the stefanberger/refactor_read_config_file branch from 45ee7d4 to 9ca0e0e Compare September 27, 2024 18:07
@stefanberger stefanberger merged commit 2e21249 into master Sep 27, 2024
5 checks passed
@stefanberger stefanberger deleted the stefanberger/refactor_read_config_file branch September 27, 2024 19:13
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.

1 participant