[libvirt] [PATCHv2 0/8] Remove VIR_STRNDUP

Ján Tomko posted 8 patches 4 years, 4 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1576070578.git.jtomko@redhat.com
docs/hacking.html.in                   |  10 +-
src/bhyve/bhyve_parse_command.c        |   6 +-
src/conf/backup_conf.c                 |   4 +-
src/conf/nwfilter_conf.c               |   4 +-
src/conf/nwfilter_params.c             |   6 +-
src/interface/interface_backend_udev.c |   8 +-
src/libvirt_private.syms               |   2 -
src/libxl/xen_common.c                 |  29 +++---
src/libxl/xen_xl.c                     |   6 +-
src/libxl/xen_xm.c                     |   9 +-
src/lxc/lxc_driver.c                   |   3 +-
src/qemu/qemu_driver.c                 |   3 +-
src/qemu/qemu_monitor_json.c           |  14 +--
src/rpc/virnetlibsshsession.c          |   7 +-
src/util/vircgroupv1.c                 |   5 +-
src/util/vircommand.c                  |   1 -
src/util/virconf.c                     |  24 ++---
src/util/viriscsi.c                    |   3 +-
src/util/virjson.c                     |  11 +-
src/util/virkeyfile.c                  |   9 +-
src/util/virsocketaddr.c               |   5 +-
src/util/virstoragefile.c              |   6 +-
src/util/virstring.c                   |  52 +---------
src/util/virstring.h                   |  69 -------------
src/util/virsysinfo.c                  |   3 +-
src/vmware/vmware_conf.c               |   6 +-
tests/virstringtest.c                  | 136 -------------------------
tools/vsh.c                            |  16 +--
28 files changed, 63 insertions(+), 394 deletions(-)
[libvirt] [PATCHv2 0/8] Remove VIR_STRNDUP
Posted by Ján Tomko 4 years, 4 months ago
Ján Tomko (8):
  Remove VIR_STRDUP usage that sneaked in in the meantime
  vsh: remove vshErrorOOM
  Remove VIR_STRNDUP usage with checked pointers
  Remove all the uses that use subtraction in their length argument
  Remove the rest of VIR_STRNDUP
  tests: delete tests for VIR_STR(N)DUP
  util: remove VIR_STRDUP and VIR_STRNDUP
  docs: hacking: document removal of VIR_STR(N)DUP

 docs/hacking.html.in                   |  10 +-
 src/bhyve/bhyve_parse_command.c        |   6 +-
 src/conf/backup_conf.c                 |   4 +-
 src/conf/nwfilter_conf.c               |   4 +-
 src/conf/nwfilter_params.c             |   6 +-
 src/interface/interface_backend_udev.c |   8 +-
 src/libvirt_private.syms               |   2 -
 src/libxl/xen_common.c                 |  29 +++---
 src/libxl/xen_xl.c                     |   6 +-
 src/libxl/xen_xm.c                     |   9 +-
 src/lxc/lxc_driver.c                   |   3 +-
 src/qemu/qemu_driver.c                 |   3 +-
 src/qemu/qemu_monitor_json.c           |  14 +--
 src/rpc/virnetlibsshsession.c          |   7 +-
 src/util/vircgroupv1.c                 |   5 +-
 src/util/vircommand.c                  |   1 -
 src/util/virconf.c                     |  24 ++---
 src/util/viriscsi.c                    |   3 +-
 src/util/virjson.c                     |  11 +-
 src/util/virkeyfile.c                  |   9 +-
 src/util/virsocketaddr.c               |   5 +-
 src/util/virstoragefile.c              |   6 +-
 src/util/virstring.c                   |  52 +---------
 src/util/virstring.h                   |  69 -------------
 src/util/virsysinfo.c                  |   3 +-
 src/vmware/vmware_conf.c               |   6 +-
 tests/virstringtest.c                  | 136 -------------------------
 tools/vsh.c                            |  16 +--
 28 files changed, 63 insertions(+), 394 deletions(-)

-- 
2.21.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCHv2 0/8] Remove VIR_STRNDUP
Posted by Michal Privoznik 4 years, 4 months ago
On 12/11/19 2:23 PM, Ján Tomko wrote:
> Ján Tomko (8):
>    Remove VIR_STRDUP usage that sneaked in in the meantime
>    vsh: remove vshErrorOOM
>    Remove VIR_STRNDUP usage with checked pointers
>    Remove all the uses that use subtraction in their length argument
>    Remove the rest of VIR_STRNDUP
>    tests: delete tests for VIR_STR(N)DUP
>    util: remove VIR_STRDUP and VIR_STRNDUP
>    docs: hacking: document removal of VIR_STR(N)DUP
> 
>   docs/hacking.html.in                   |  10 +-
>   src/bhyve/bhyve_parse_command.c        |   6 +-
>   src/conf/backup_conf.c                 |   4 +-
>   src/conf/nwfilter_conf.c               |   4 +-
>   src/conf/nwfilter_params.c             |   6 +-
>   src/interface/interface_backend_udev.c |   8 +-
>   src/libvirt_private.syms               |   2 -
>   src/libxl/xen_common.c                 |  29 +++---
>   src/libxl/xen_xl.c                     |   6 +-
>   src/libxl/xen_xm.c                     |   9 +-
>   src/lxc/lxc_driver.c                   |   3 +-
>   src/qemu/qemu_driver.c                 |   3 +-
>   src/qemu/qemu_monitor_json.c           |  14 +--
>   src/rpc/virnetlibsshsession.c          |   7 +-
>   src/util/vircgroupv1.c                 |   5 +-
>   src/util/vircommand.c                  |   1 -
>   src/util/virconf.c                     |  24 ++---
>   src/util/viriscsi.c                    |   3 +-
>   src/util/virjson.c                     |  11 +-
>   src/util/virkeyfile.c                  |   9 +-
>   src/util/virsocketaddr.c               |   5 +-
>   src/util/virstoragefile.c              |   6 +-
>   src/util/virstring.c                   |  52 +---------
>   src/util/virstring.h                   |  69 -------------
>   src/util/virsysinfo.c                  |   3 +-
>   src/vmware/vmware_conf.c               |   6 +-
>   tests/virstringtest.c                  | 136 -------------------------
>   tools/vsh.c                            |  16 +--
>   28 files changed, 63 insertions(+), 394 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list