Thanks for your reply. I will add one empty line in 1/8 patch in v2. Regards Zhiqiang Liu
On 2020/11/21 0:45, Jeff Moyer wrote: > Zhiqiang Liu <[email protected]> writes: > >> Recently, we use Coverity to analysis the ndctl package. >> Several issues should be resolved to make Coverity happy. >> >> lihaotian9 (8): >> namespace: check whether pfn|dax|btt is NULL in setup_namespace >> lib/libndctl: fix memory leakage problem in add_bus >> libdaxctl: fix memory leakage in add_dax_region() >> dimm: fix potential fd leakage in dimm_action() >> util/help: check whether strdup returns NULL in exec_man_konqueror >> lib/inject: check whether cmd is created successfully >> libndctl: check whether ndctl_btt_get_namespace returns NULL in >> callers >> namespace: check whether seed is NULL in validate_namespace_options >> >> daxctl/lib/libdaxctl.c | 3 +++ >> ndctl/dimm.c | 12 +++++++----- >> ndctl/lib/inject.c | 8 ++++++++ >> ndctl/lib/libndctl.c | 1 + >> ndctl/namespace.c | 23 ++++++++++++++++++----- >> test/libndctl.c | 16 +++++++++++----- >> test/parent-uuid.c | 2 +- >> util/help.c | 8 +++++++- >> util/json.c | 3 +++ >> 9 files changed, 59 insertions(+), 17 deletions(-) > > Except for the minor nit I pointed out, for the series: > > Acked-by: Jeff Moyer <[email protected]> > > > . > _______________________________________________ Linux-nvdimm mailing list -- [email protected] To unsubscribe send an email to [email protected]
