-
-
Notifications
You must be signed in to change notification settings - Fork 59
Make qfile-dom0-unpacker validate values set via environment variables #8882
Copy link
Copy link
Labels
C: coreThis issue pertains to a Qubes core component.This issue pertains to a Qubes core component.P: defaultPriority: default. Default priority for new issues, to be replaced given sufficient information.Priority: default. Default priority for new issues, to be replaced given sufficient information.good first issueThis is a recommended issue for first-time contributors.This is a recommended issue for first-time contributors.pr submittedA pull request has been submitted for this issue.A pull request has been submitted for this issue.
Metadata
Metadata
Assignees
Labels
C: coreThis issue pertains to a Qubes core component.This issue pertains to a Qubes core component.P: defaultPriority: default. Default priority for new issues, to be replaced given sufficient information.Priority: default. Default priority for new issues, to be replaced given sufficient information.good first issueThis is a recommended issue for first-time contributors.This is a recommended issue for first-time contributors.pr submittedA pull request has been submitted for this issue.A pull request has been submitted for this issue.
Type
Fields
Give feedbackNo fields configured for Feature.
How to file a helpful issue
The problem you're addressing (if any)
I want to be able to use qfile-agent in dom0 and when dealing with large files, I need to modify
UPDATES_MAX_BYTESandUPDATES_MAX_FILES. Unfortunately, it does not validate the variables. If the variable is invalid,atoll()will convert it to0, thus no limit is set instead of failing.The solution you'd like
Fail when the user set a value different than zero and
bytes_limitorfiles_limitis converted to0.The value to a user, and who that user might be
In order to complete
It must fail on invalid user input instead of allowing the user mistake to unlimit the values.