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