g_strdup(NULL) returns NULL; simplify
Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
This commit is contained in:
		
							parent
							
								
									6528499fa4
								
							
						
					
					
						commit
						c64f50d1e2
					
				@ -85,11 +85,7 @@ VirtIODevice *virtio_9p_init(DeviceState *dev, V9fsConf *conf)
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    s->ctx.export_flags = fse->export_flags;
 | 
			
		||||
    if (fse->path) {
 | 
			
		||||
    s->ctx.fs_root = g_strdup(fse->path);
 | 
			
		||||
    } else {
 | 
			
		||||
        s->ctx.fs_root = NULL;
 | 
			
		||||
    }
 | 
			
		||||
    s->ctx.exops.get_st_gen = NULL;
 | 
			
		||||
    len = strlen(conf->tag);
 | 
			
		||||
    if (len > MAX_TAG_LEN - 1) {
 | 
			
		||||
 | 
			
		||||
@ -18,10 +18,8 @@ void blkconf_serial(BlockConf *conf, char **serial)
 | 
			
		||||
    if (!*serial) {
 | 
			
		||||
        /* try to fall back to value set with legacy -drive serial=... */
 | 
			
		||||
        dinfo = drive_get_by_blockdev(conf->bs);
 | 
			
		||||
        if (dinfo->serial) {
 | 
			
		||||
        *serial = g_strdup(dinfo->serial);
 | 
			
		||||
    }
 | 
			
		||||
    }
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
int blkconf_geometry(BlockConf *conf, int *ptrans,
 | 
			
		||||
 | 
			
		||||
@ -566,9 +566,7 @@ static int net_init_nic(const NetClientOptions *opts, const char *name,
 | 
			
		||||
        assert(peer);
 | 
			
		||||
        nd->netdev = peer;
 | 
			
		||||
    }
 | 
			
		||||
    if (name) {
 | 
			
		||||
    nd->name = g_strdup(name);
 | 
			
		||||
    }
 | 
			
		||||
    if (nic->has_model) {
 | 
			
		||||
        nd->model = g_strdup(nic->model);
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
@ -225,12 +225,8 @@ Slirp *slirp_init(int restricted, struct in_addr vnetwork,
 | 
			
		||||
        pstrcpy(slirp->client_hostname, sizeof(slirp->client_hostname),
 | 
			
		||||
                vhostname);
 | 
			
		||||
    }
 | 
			
		||||
    if (tftp_path) {
 | 
			
		||||
    slirp->tftp_prefix = g_strdup(tftp_path);
 | 
			
		||||
    }
 | 
			
		||||
    if (bootfile) {
 | 
			
		||||
    slirp->bootp_filename = g_strdup(bootfile);
 | 
			
		||||
    }
 | 
			
		||||
    slirp->vdhcp_startaddr = vdhcp_start;
 | 
			
		||||
    slirp->vnameserver_addr = vnameserver;
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
@ -643,9 +643,7 @@ static void opt_set(QemuOpts *opts, const char *name, const char *value,
 | 
			
		||||
        QTAILQ_INSERT_TAIL(&opts->head, opt, next);
 | 
			
		||||
    }
 | 
			
		||||
    opt->desc = desc;
 | 
			
		||||
    if (value) {
 | 
			
		||||
    opt->str = g_strdup(value);
 | 
			
		||||
    }
 | 
			
		||||
    qemu_opt_parse(opt, &local_err);
 | 
			
		||||
    if (error_is_set(&local_err)) {
 | 
			
		||||
        error_propagate(errp, local_err);
 | 
			
		||||
@ -792,9 +790,7 @@ QemuOpts *qemu_opts_create(QemuOptsList *list, const char *id,
 | 
			
		||||
        }
 | 
			
		||||
    }
 | 
			
		||||
    opts = g_malloc0(sizeof(*opts));
 | 
			
		||||
    if (id) {
 | 
			
		||||
    opts->id = g_strdup(id);
 | 
			
		||||
    }
 | 
			
		||||
    opts->list = list;
 | 
			
		||||
    loc_save(&opts->loc);
 | 
			
		||||
    QTAILQ_INIT(&opts->head);
 | 
			
		||||
 | 
			
		||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user