Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upget rid of system() in vm-file-editor.c #358
Comments
marmarek
assigned
rootkovska
Mar 8, 2015
marmarek
added this to the Release 1 Beta 3 milestone
Mar 8, 2015
marmarek
added
bug
C: core
P: major
labels
Mar 8, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
|
Modified by joanna on 25 Sep 2011 10:51 UTC |
marmarek
added
P: minor
and removed
P: major
labels
Mar 8, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
|
Modified by marmarek on 17 Oct 2011 12:09 UTC |
marmarek
assigned
marmarek
and unassigned
rootkovska
Mar 8, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
marmarek
Mar 8, 2015
Member
Comment by marmarek on 17 Oct 2011 22:14 UTC
http://git.qubes-os.org/gitweb/?p=marmarek/core.git;a=commit;h=e6bbc83e0bf6d176d692eef51719672cd34011b1
|
Comment by marmarek on 17 Oct 2011 22:14 UTC |
marmarek
closed this
Mar 8, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
marmarek commentedMar 8, 2015
Reported by rafal on 16 Sep 2011 14:14 UTC
Replace with fork/close/exec.
Not really a big issue, more of taste - we already assume that granting OpenInVM means granting control over target vm.
Migrated-From: https://wiki.qubes-os.org/ticket/358