[libvirt] [PATCH v2 00/11] remove repetition of URI path validation

Daniel Henrique Barboza danielhb413 at gmail.com
Mon Sep 23 20:44:12 UTC 2019


This is a code repetition that I crossed a few times, then
I noticed that Cole Robinson suggested a solution for it
in the wiki. Here it is.

changes from v1:
- handle QEMU and vbox cases separately inside the validation
function

v1: https://www.redhat.com/archives/libvir-list/2019-September/msg00983.html

Daniel Henrique Barboza (11):
  src/driver.c: add virConnectValidateURIPath()
  interface_backend_netcf.c: use virConnectValidateURIPath()
  interface_backend_udev.c: use virConnectValidateURIPath()
  bridge_driver.c: virConnectValidateURIPath()
  node_device_driver.c: use virConnectValidateURIPath()
  secret_driver.c: use virConnectValidateURIPath()
  storage_driver.c: use virConnectValidateURIPath()
  driver.c: change URI validation to handle QEMU and vbox case
  qemu_driver.c: use virConnectValidateURIPath()
  vbox_common.c: use virConnectValidateURIPath()
  vbox_driver.c: use virConnectValidateURIPath()

 src/driver.c                            | 40 +++++++++++++++++++++++++
 src/driver.h                            |  4 +++
 src/interface/interface_backend_netcf.c | 17 ++---------
 src/interface/interface_backend_udev.c  | 17 ++---------
 src/libvirt_private.syms                |  1 +
 src/network/bridge_driver.c             | 19 +++---------
 src/node_device/node_device_driver.c    | 17 ++---------
 src/qemu/qemu_driver.c                  | 20 +++----------
 src/secret/secret_driver.c              | 17 ++---------
 src/storage/storage_driver.c            | 17 ++---------
 src/vbox/vbox_common.c                  | 16 ++--------
 src/vbox/vbox_driver.c                  | 16 ++--------
 12 files changed, 67 insertions(+), 134 deletions(-)

-- 
2.21.0




More information about the libvir-list mailing list