Re: [Qemu-devel] [PATCH] 9pfs: fail local_statfs() earlier

2017-03-03 Thread Philippe Mathieu-Daudé
On 03/03/2017 03:03 PM, Greg Kurz wrote: If we cannot open the given path, we can return right away instead of passing -1 to fstatfs() and close(). This will make Coverity happy. (Coverity issue CID1371729) Signed-off-by: Greg Kurz Reviewed-by: Philippe Mathieu-Daudé --- hw/9pfs/9p-local

Re: [Qemu-devel] [PATCH] 9pfs: fail local_statfs() earlier

2017-03-03 Thread Eric Blake
On 03/03/2017 12:03 PM, Greg Kurz wrote: > If we cannot open the given path, we can return right away instead of > passing -1 to fstatfs() and close(). This will make Coverity happy. > > (Coverity issue CID1371729) > > Signed-off-by: Greg Kurz > --- > hw/9pfs/9p-local.c |3 +++ > 1 file cha

Re: [Qemu-devel] [PATCH] 9pfs: fail local_statfs() earlier

2017-03-03 Thread Daniel P. Berrange
On Fri, Mar 03, 2017 at 07:03:47PM +0100, Greg Kurz wrote: > If we cannot open the given path, we can return right away instead of > passing -1 to fstatfs() and close(). This will make Coverity happy. > > (Coverity issue CID1371729) > > Signed-off-by: Greg Kurz > --- > hw/9pfs/9p-local.c |3

[Qemu-devel] [PATCH] 9pfs: fail local_statfs() earlier

2017-03-03 Thread Greg Kurz
If we cannot open the given path, we can return right away instead of passing -1 to fstatfs() and close(). This will make Coverity happy. (Coverity issue CID1371729) Signed-off-by: Greg Kurz --- hw/9pfs/9p-local.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/9pfs/9p-local.c b/hw