On 1/4/24 01:16, Artem Chernyshev wrote:
> Several functions was modified to become invariant. Change
> their type to void and remove unnecessary checks of their
> return values.
> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
> 
> Artem Chernyshev (7):
>   cpu: : virCPUx86DataAddItem() to void
>   rpc: virnetserver: virNetServerAddService() to void
>   lxc: virLXCControllerAddConsole() to void
>   conf: virDomainNetUpdate() to void
>   virprocess: virProcessGetNamespaces() to void
>   node_device: udevTranslatePCIIds() to void
>   node_device: udevGetStringSysfsAttr() to void
> 
>  src/conf/domain_conf.c             |   3 +-
>  src/conf/domain_conf.h             |   2 +-
>  src/cpu/cpu_x86.c                  | 179 +++++++++++------------------
>  src/cpu/cpu_x86.h                  |   4 +-
>  src/libvirt-lxc.c                  |   5 +-
>  src/libxl/libxl_capabilities.c     |  13 +--
>  src/lxc/lxc_controller.c           |   9 +-
>  src/lxc/lxc_driver.c               |   6 +-
>  src/node_device/node_device_udev.c |  69 ++++-------
>  src/qemu/qemu_capabilities.c       |   3 +-
>  src/qemu/qemu_driver.c             |   3 +-
>  src/qemu/qemu_monitor_json.c       |   7 +-
>  src/qemu/qemu_process.c            |   3 +-
>  src/rpc/virnetserver.c             |  25 +---
>  src/rpc/virnetserver.h             |   2 +-
>  src/util/virprocess.c              |   4 +-
>  src/util/virprocess.h              |   2 +-
>  tests/virnetdaemontest.c           |   6 +-
>  18 files changed, 118 insertions(+), 227 deletions(-)
> 

Fixed small nits I've raised during review and pushed.

Congratulations on your first libvirt contribution!

Reviewed-by: Michal Privoznik <[email protected]>

Michal
_______________________________________________
Devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to