aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLi Qiang <liq3ea@163.com>2020-08-19 07:43:09 -0700
committerLaurent Vivier <laurent@vivier.eu>2020-09-23 19:08:29 +0200
commit986d4f78742b7419f01d8ede242abde6f2f74cf2 (patch)
treeea946925f78744751be2528c2d6fa3fd9640569a
parent3e8529dd8554d77467734db6f84b3619eeeb6895 (diff)
downloadqemu-986d4f78742b7419f01d8ede242abde6f2f74cf2.zip
qemu-986d4f78742b7419f01d8ede242abde6f2f74cf2.tar.gz
qemu-986d4f78742b7419f01d8ede242abde6f2f74cf2.tar.bz2
virtio: vdpa: omit check return of g_malloc
If g_malloc fails, the application will be terminated. No need to check the return value of g_malloc. Signed-off-by: Li Qiang <liq3ea@163.com> Reviewed-by: Laurent Vivier <lvivier@redhat.com> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20200819144309.67579-1-liq3ea@163.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
-rw-r--r--hw/virtio/vhost-vdpa.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
index e123837..97f4b2e 100644
--- a/hw/virtio/vhost-vdpa.c
+++ b/hw/virtio/vhost-vdpa.c
@@ -320,10 +320,8 @@ static int vhost_vdpa_set_config(struct vhost_dev *dev, const uint8_t *data,
struct vhost_vdpa_config *config;
int ret;
unsigned long config_size = offsetof(struct vhost_vdpa_config, buf);
+
config = g_malloc(size + config_size);
- if (config == NULL) {
- return -1;
- }
config->off = offset;
config->len = size;
memcpy(config->buf, data, size);
@@ -340,9 +338,6 @@ static int vhost_vdpa_get_config(struct vhost_dev *dev, uint8_t *config,
int ret;
v_config = g_malloc(config_len + config_size);
- if (v_config == NULL) {
- return -1;
- }
v_config->len = config_len;
v_config->off = 0;
ret = vhost_vdpa_call(dev, VHOST_VDPA_GET_CONFIG, v_config);