Re: [ndctl PATCH] ndctl: fix ordering of closing vs invalidating flush_fd
On 04/16/2018 10:03 AM, Vishal Verma wrote: > Static analysis warns that we may be closing a negative 'fd'. Fix this > by closing before invalidating the flush_fd with a -1 in add_region() > > Fixes: commit c64cc15 ("ndctl: add support in libndctl to provide deep flush") > Cc: Dave Jiang> Signed-off-by: Vishal Verma Good catch! Acked-by: Dave Jiang > --- > ndctl/lib/libndctl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ndctl/lib/libndctl.c b/ndctl/lib/libndctl.c > index ad5ab9a..59ea82a 100644 > --- a/ndctl/lib/libndctl.c > +++ b/ndctl/lib/libndctl.c > @@ -1929,8 +1929,8 @@ static void *add_region(void *parent, int id, const > char *region_base) > > perm = strtol(buf, NULL, 0); > if (perm == 0) { > - region->flush_fd = -1; > close(region->flush_fd); > + region->flush_fd = -1; > } > > out: > ___ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm
[ndctl PATCH] ndctl: fix ordering of closing vs invalidating flush_fd
Static analysis warns that we may be closing a negative 'fd'. Fix this by closing before invalidating the flush_fd with a -1 in add_region() Fixes: commit c64cc15 ("ndctl: add support in libndctl to provide deep flush") Cc: Dave JiangSigned-off-by: Vishal Verma --- ndctl/lib/libndctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ndctl/lib/libndctl.c b/ndctl/lib/libndctl.c index ad5ab9a..59ea82a 100644 --- a/ndctl/lib/libndctl.c +++ b/ndctl/lib/libndctl.c @@ -1929,8 +1929,8 @@ static void *add_region(void *parent, int id, const char *region_base) perm = strtol(buf, NULL, 0); if (perm == 0) { - region->flush_fd = -1; close(region->flush_fd); + region->flush_fd = -1; } out: -- 2.14.3 ___ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm