[Virtio-fs] [PATCH] virtiofsd: move assert just after calloc()
piaojun
piaojun at huawei.com
Mon Aug 5 01:53:23 UTC 2019
It's better to assert NULL pointer just after calloc().
Signed-off-by: Jun Piao <piaojun at huawei.com>
---
contrib/virtiofsd/fuse_virtio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/virtiofsd/fuse_virtio.c b/contrib/virtiofsd/fuse_virtio.c
index d543c6d..8a4f627 100644
--- a/contrib/virtiofsd/fuse_virtio.c
+++ b/contrib/virtiofsd/fuse_virtio.c
@@ -748,9 +748,9 @@ static void fv_queue_set_started(VuDev *dev, int qidx, bool started)
}
if (!vud->qi[qidx]) {
vud->qi[qidx] = calloc(sizeof(struct fv_QueueInfo), 1);
+ assert(vud->qi[qidx]);
vud->qi[qidx]->virtio_dev = vud;
vud->qi[qidx]->qidx = qidx;
- assert(vud->qi[qidx]);
} else {
/* Shouldn't have been started */
assert(vud->qi[qidx]->kick_fd == -1);
--
More information about the Virtio-fs
mailing list