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.

Sign up for GitHub

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

Jump to bottom

VxMark: Does not check blank pages are actually blank #5213

Closed
arsalansufi opened this issue Aug 5, 2024 · 1 comment
Closed

VxMark: Does not check blank pages are actually blank #5213

arsalansufi opened this issue Aug 5, 2024 · 1 comment
Labels
triage

Comments

@arsalansufi
Copy link
Contributor

arsalansufi commented Aug 5, 2024 •
edited by adghayes
Loading

VxMark: Machine accepts an HMPB as a "blank page." What are the security implications of this? Should we be checking the page is blank?

Issue automatically created from Slack feedback reported by caroline:
https://votingworks.slack.com/archives/CJU9MSC6S/p1722455194903769?thread_ts=1722455194.903769&cid=CJU9MSC6S
@adghayes adghayes changed the title VxMark: Machine accepts a HMPB non-thermal page as a "blank page" VxMark: Does not check blank pages are actually blank Aug 5, 2024
@adghayes
Copy link
Collaborator

adghayes commented Aug 7, 2024

We are OK with accepting non-blank pages here - not a meaningful malicious path and a regular user can just insert new paper.

@adghayes adghayes closed this as not planned Won't fix, can't repro, duplicate, stale Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arsalansufi @adghayes