Message ID | 1299347533-17047-3-git-send-email-aneesh.kumar@linux.vnet.ibm.com |
---|---|
State | New |
Headers | show |
On Sat, Mar 5, 2011 at 5:52 PM, Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> wrote: > @@ -1873,7 +1873,7 @@ static void v9fs_post_lcreate(V9fsState *s, V9fsLcreateState *vs, int err) > vs->fidp->fsmap.fid_type = P9_FID_NONE; > err = -errno; > if (vs->fidp->fsmap.fs.fd > 0) { For completeness: vs->fdip->fsmap.fs.fd >= 0 Stefan
On Sun, 13 Mar 2011 16:10:20 +0000, Stefan Hajnoczi <stefanha@gmail.com> wrote: > On Sat, Mar 5, 2011 at 5:52 PM, Aneesh Kumar K.V > <aneesh.kumar@linux.vnet.ibm.com> wrote: > > @@ -1873,7 +1873,7 @@ static void v9fs_post_lcreate(V9fsState *s, V9fsLcreateState *vs, int err) > > vs->fidp->fsmap.fid_type = P9_FID_NONE; > > err = -errno; > > if (vs->fidp->fsmap.fs.fd > 0) { > > For completeness: vs->fdip->fsmap.fs.fd >= 0 > Will update -aneesh
diff --git a/hw/9pfs/virtio-9p.c b/hw/9pfs/virtio-9p.c index 811ac38..c4b0198 100644 --- a/hw/9pfs/virtio-9p.c +++ b/hw/9pfs/virtio-9p.c @@ -1873,7 +1873,7 @@ static void v9fs_post_lcreate(V9fsState *s, V9fsLcreateState *vs, int err) vs->fidp->fsmap.fid_type = P9_FID_NONE; err = -errno; if (vs->fidp->fsmap.fs.fd > 0) { - close(vs->fidp->fsmap.fs.fd); + v9fs_do_close(s, vs->fidp->fsmap.fs.fd); } } @@ -2533,7 +2533,7 @@ static void v9fs_create_post_fstat(V9fsState *s, V9fsCreateState *vs, int err) { if (err) { vs->fidp->fsmap.fid_type = P9_FID_NONE; - close(vs->fidp->fsmap.fs.fd); + v9fs_do_close(s, vs->fidp->fsmap.fs.fd); err = -errno; } v9fs_post_create(s, vs, err);
we should use the local abstraction instead of directly calling close. Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> --- hw/9pfs/virtio-9p.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)