Re: [libvirt PATCH 00/10] virHashNew refactorings - part V

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a %A in %Y, Tim Wiederhake wrote:
> "virHashNew" cannot return NULL, yet we check for NULL in various places.
> 
> See https://listman.redhat.com/archives/libvir-list/2021-July/msg00074.html.
> 
> Tim Wiederhake (10):
>   conf: Add AUTOPTR_CLEANUP_FUNC for virNWFilterVarValue
>   virNWFilterParseParamAttributes: `virHashNew` cannot return NULL
>   virNWFilterParseParamAttributes: Iterate over "element" children
>   virNWFilterParseParamAttributes: Remove tautological `if`
>   virNWFilterParseParamAttributes: Use automatic memory management
>   virNWFilterParseParamAttributes: Simplify loop body
>   virNWFilterParseParamAttributes: Remove superfluous `goto`s
>   virNWFilterDoInstantiate: `virHashNew` cannot return NULL
>   virNWFilterIPAddrMapInit: `virHashNew` cannot return NULL
>   virNWFilterLearnInit: `virHashNew` cannot return NULL
> 
>  src/conf/nwfilter_ipaddrmap.c          |  3 --
>  src/conf/nwfilter_params.c             | 70 +++++++++-----------------
>  src/conf/nwfilter_params.h             |  2 +
>  src/nwfilter/nwfilter_gentech_driver.c |  5 --
>  src/nwfilter/nwfilter_learnipaddr.c    |  8 ---
>  5 files changed, 26 insertions(+), 62 deletions(-)
> 

Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux