/qubes-issues

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

qvm-prefs: verify 'pcidevs' property #1271

Closed
marmarek opened this Issue Oct 5, 2015 · 2 comments

Comments

Assignees
No one assigned
Labels
Projects
None yet
Milestone
  Release 3.1
1 participant
@marmarek
@marmarek
Member

marmarek commented Oct 5, 2015

'pcidevs' property should be verified at set time - it shouldn't be possible to
assign non-existent devices. Also the current parsing code is wrong - it gets
the property value and eval() it to get a list.
@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Oct 5, 2015

Member

Alternatively setting this property using qvm-prefs can be disabled, leaving
only qvm-pci tool.
Member

marmarek commented Oct 5, 2015

Alternatively setting this property using qvm-prefs can be disabled, leaving
only qvm-pci tool.

@marmarek marmarek added bug C: core P: minor labels Oct 5, 2015

@marmarek marmarek modified the milestones: Far in the future, Release 3.1 Oct 5, 2015

@marmarek marmarek added the wontfix label Feb 11, 2016

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Feb 11, 2016

Member

In R4.0 it won't be an issue any longer - qvm-prefs doesn't support setting pcidevs anymore.
Member

marmarek commented Feb 11, 2016

In R4.0 it won't be an issue any longer - qvm-prefs doesn't support setting pcidevs anymore.

@marmarek marmarek closed this Feb 11, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment