mbox series

[ndctl,0/4] static analysis fixes

Message ID 20190510190839.29637-1-vishal.l.verma@intel.com (mailing list archive)
Headers show
Series static analysis fixes | expand

Message

Verma, Vishal L May 10, 2019, 7:08 p.m. UTC 
This series just contains a handful of static analysis fixes in
preparation for the v65 release.

Vishal Verma (4):
  ndctl/namespace.c: fix resource leak
  ndctl/namespace.c: fix an unchecked return value
  libndctl: Fix an unhandled return in ndctl_bus_poll_scrub_completion()
  ndctl/namespace.c: Fix a potential integer overflow

 ndctl/lib/libndctl.c |  2 ++
 ndctl/namespace.c    | 28 ++++++++++++++++------------
 2 files changed, 18 insertions(+), 12 deletions(-)

Comments

Dan Williams May 10, 2019, 7:58 p.m. UTC | #1 
On Fri, May 10, 2019 at 12:08 PM Vishal Verma <vishal.l.verma@intel.com> wrote:
>
> This series just contains a handful of static analysis fixes in
> preparation for the v65 release.
>
> Vishal Verma (4):
>   ndctl/namespace.c: fix resource leak
>   ndctl/namespace.c: fix an unchecked return value
>   libndctl: Fix an unhandled return in ndctl_bus_poll_scrub_completion()
>   ndctl/namespace.c: Fix a potential integer overflow

These all look good to me.

Reviewed-by: Dan Williams <dan.j.williams@intel.com>