OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

virtio: vdpa: omit check return of g_malloc

Browse Source 
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>
This commit is contained in:
Li Qiang 2020-08-19 07:43:09 -07:00 committed by Laurent Vivier
parent 3e8529dd85
commit 986d4f7874
1 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hw/virtio/vhost-vdpa.c
View File

@ -320,10 +320,8 @@ static int vhost_vdpa_set_config(struct vhost_dev *dev, const uint8_t *data,
struct vhost_vdpa_config *config; struct vhost_vdpa_config *config;
int ret; int ret;
unsigned long config_size = offsetof(struct vhost_vdpa_config, buf); unsigned long config_size = offsetof(struct vhost_vdpa_config, buf);
config = g_malloc(size + config_size); config = g_malloc(size + config_size);
if (config == NULL) {
return -1;
}
config->off = offset; config->off = offset;
config->len = size; config->len = size;
memcpy(config->buf, data, 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; int ret;
v_config = g_malloc(config_len + config_size); v_config = g_malloc(config_len + config_size);
if (v_config == NULL) {
return -1;
}
v_config->len = config_len; v_config->len = config_len;
v_config->off = 0; v_config->off = 0;
ret = vhost_vdpa_call(dev, VHOST_VDPA_GET_CONFIG, v_config); ret = vhost_vdpa_call(dev, VHOST_VDPA_GET_CONFIG, v_config);