[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v8 00/11] Add a standard authorization framework
From: |
Daniel P . Berrangé |
Subject: |
[Qemu-devel] [PATCH v8 00/11] Add a standard authorization framework |
Date: |
Fri, 15 Feb 2019 15:56:58 +0000 |
An update to
v2: https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg04469.html
v3: https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg05660.html
v4: https://lists.gnu.org/archive/html/qemu-devel/2018-08/msg02961.html
v5: https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg01613.html
v6: https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg04435.html
v7: https://lists.gnu.org/archive/html/qemu-devel/2018-11/msg06616.html
The current network services now support encryption via TLS and in some
cases support authentication via SASL. In cases where SASL is not
available, x509 client certificates can be used as a crude authorization
scheme, but using a sub-CA and controlling who you give certs to. In
general this is not very flexible though, so this series introduces a
new standard authorization framework.
It comes with four initial authorization mechanisms
- Simple - an exact username match. This is useful when there is
exactly one user that is known to connect. For example when live
migrating from one QEMU to another with TLS, libvirt would use
the simple scheme to whitelist the TLS cert of the source QEMU.
- List - an full access control list, with optional regex matching.
This is more flexible and is used to provide 100% backcompat with
the existing HMP ACL commands. The caveat is that we can't create
these via the CLI -object arg yet.
- ListFile - the same as List, but with the rules stored in JSON
format in an external file. This avoids the -object limitation
while also allowing the admin to change list entries on the file.
QEMU uses inotify to notice these changes and auto-reload the
file contents. This is likely a good default choice for most
network services, if the "simple" mechanism isn't sufficient.
- PAM - delegate the username lookup to a PAM module, which opens
the door to many options including things like SQL/LDAP lookups.
A later series that follows will integrate this framework into the VNC,
NBD, migration, and character device servers.
Changed in v7:
- Fix missing case QFILE_MONITOR_EVENT_ATTRIBUTES in mtp dev
- Remove obsolete CONFIG_FNMATCH in monitor code
- Remove obsolete fnmatch probe in configure
- Fix QMP monitor example docs (Philippe)
Changed in v6:
- Change file monitor object to allow multiple instances
instead of being a singleton
- Use stubs when inotify is not available
- Use GArray for managing watches
- Report errno when failing to read from inotify
- Check for closed inotify FD before reading
- Monitor file attribute changes to detect chmod/chown/etc
- Unregister event loop FD handler when watches are removed
- Fix mutex unlock call when removing watches
- Remove file monitor watch when free'ing MTP object
- Rename MTP watchfd field to watchid
- Free file monitor when deleting MTP device
- Use error_report instead of fprintf for MTP errors
- Use GPattern instead of fnmatch
- Use enum _str() helper functions
- Add unit/functional test for inotify file monitor
- Add unit test for simple authz impl
- Add unit test for list file authz impl
- Add unit test for PAM authz impl
Changed in v6:
- Fix consistency of object variable names
- Fix command line example quoting
- Use 'false' not '0' for bool return values
- Remove empty 'complete' methods
- Fix leak of rules in finalizer
- Fix missing {} around conditional
- Refactor rule refresh to use goto & invert conditional
Changed in v5:
- Rebase to latest git master
Changed in v4:
- Rebase to latest git master
Changed in v3:
- Added docs for object types in qemu-options.hx
- Added example CLI syntax in header files
- Improved commit messages
Changed in v2:
- Switch to a global shared instance of the file monitor so only
a single inotify file descriptor is required
- Require all watches to be registered against directories. File
watches are useless in Linux, since they are tied to inodes, and
so stop working when editors save by doing a tmpfile + rename
dance.
- Change auth list impl to use a directory based watch instead
of filename
- Put MTP const-ness fixes in separate patch
- Split QOM change off into separate patch
- Fix conditionals on Win32 build
Daniel P. Berrangé (11):
util: add helper APIs for dealing with inotify in portable manner
qom: don't require user creatable objects to be registered
hw/usb: don't set IN_ISDIR for inotify watch in MTP driver
hw/usb: fix const-ness for string params in MTP driver
hw/usb: switch MTP to use new inotify APIs
authz: add QAuthZ object as an authorization base class
authz: add QAuthZSimple object type for easy whitelist auth checks
authz: add QAuthZList object type for an access control list
authz: add QAuthZListFile object type for a file access control list
authz: add QAuthZPAM object type for authorizing using PAM
authz: delete existing ACL implementation
MAINTAINERS | 15 +
Makefile | 10 +-
Makefile.objs | 10 +-
Makefile.target | 2 +
authz/Makefile.objs | 7 +
authz/base.c | 82 ++++
authz/list.c | 271 +++++++++++++
authz/listfile.c | 283 ++++++++++++++
authz/pamacct.c | 149 +++++++
authz/simple.c | 115 ++++++
authz/trace-events | 18 +
configure | 54 ++-
crypto/tlssession.c | 35 +-
crypto/trace-events | 2 +-
hw/usb/dev-mtp.c | 281 ++++++--------
hw/usb/trace-events | 2 +-
include/authz/base.h | 112 ++++++
include/authz/list.h | 106 +++++
include/authz/listfile.h | 111 ++++++
include/authz/pamacct.h | 100 +++++
include/authz/simple.h | 84 ++++
include/qemu/acl.h | 66 ----
include/qemu/filemonitor.h | 128 ++++++
monitor.c | 179 ++++++---
qapi/authz.json | 58 +++
qapi/qapi-schema.json | 1 +
qemu-options.hx | 105 +++++
qom/object.c | 12 +-
qom/object_interfaces.c | 16 +-
tests/Makefile.include | 16 +-
tests/test-authz-list.c | 159 ++++++++
tests/test-authz-listfile.c | 195 ++++++++++
tests/test-authz-pam.c | 124 ++++++
tests/test-authz-simple.c | 50 +++
tests/test-crypto-tlssession.c | 15 +-
tests/test-io-channel-tls.c | 16 +-
tests/test-util-filemonitor.c | 685 +++++++++++++++++++++++++++++++++
ui/vnc-auth-sasl.c | 23 +-
ui/vnc-auth-sasl.h | 5 +-
ui/vnc-auth-vencrypt.c | 2 +-
ui/vnc-ws.c | 2 +-
ui/vnc.c | 37 +-
ui/vnc.h | 4 +-
util/Makefile.objs | 4 +-
util/acl.c | 179 ---------
util/filemonitor-inotify.c | 338 ++++++++++++++++
util/filemonitor-stub.c | 59 +++
util/trace-events | 9 +
48 files changed, 3773 insertions(+), 563 deletions(-)
create mode 100644 authz/Makefile.objs
create mode 100644 authz/base.c
create mode 100644 authz/list.c
create mode 100644 authz/listfile.c
create mode 100644 authz/pamacct.c
create mode 100644 authz/simple.c
create mode 100644 authz/trace-events
create mode 100644 include/authz/base.h
create mode 100644 include/authz/list.h
create mode 100644 include/authz/listfile.h
create mode 100644 include/authz/pamacct.h
create mode 100644 include/authz/simple.h
delete mode 100644 include/qemu/acl.h
create mode 100644 include/qemu/filemonitor.h
create mode 100644 qapi/authz.json
create mode 100644 tests/test-authz-list.c
create mode 100644 tests/test-authz-listfile.c
create mode 100644 tests/test-authz-pam.c
create mode 100644 tests/test-authz-simple.c
create mode 100644 tests/test-util-filemonitor.c
delete mode 100644 util/acl.c
create mode 100644 util/filemonitor-inotify.c
create mode 100644 util/filemonitor-stub.c
--
2.20.1
- [Qemu-devel] [PATCH v8 00/11] Add a standard authorization framework,
Daniel P . Berrangé <=
- [Qemu-devel] [PATCH v8 04/11] hw/usb: fix const-ness for string params in MTP driver, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 03/11] hw/usb: don't set IN_ISDIR for inotify watch in MTP driver, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 02/11] qom: don't require user creatable objects to be registered, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 05/11] hw/usb: switch MTP to use new inotify APIs, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 06/11] authz: add QAuthZ object as an authorization base class, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 07/11] authz: add QAuthZSimple object type for easy whitelist auth checks, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 01/11] util: add helper APIs for dealing with inotify in portable manner, Daniel P . Berrangé, 2019/02/15
- [Qemu-devel] [PATCH v8 08/11] authz: add QAuthZList object type for an access control list, Daniel P . Berrangé, 2019/02/15