提交 771f1339 编写于 作者: A aliguori

Fix error handling in net_client_init() (Mark McLoughlin)

We weren't freeing the name string everywhere.
Signed-off-by: NMark McLoughlin <markmc@redhat.com>
Signed-off-by: NAnthony Liguori <aliguori@us.ibm.com>


git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@7144 c046a42c-6fe2-441c-8c8c-71466251a162
上级 efb816c9
...@@ -1610,7 +1610,8 @@ int net_client_init(const char *device, const char *p) ...@@ -1610,7 +1610,8 @@ int net_client_init(const char *device, const char *p)
if (idx == -1 || nb_nics >= MAX_NICS) { if (idx == -1 || nb_nics >= MAX_NICS) {
fprintf(stderr, "Too Many NICs\n"); fprintf(stderr, "Too Many NICs\n");
return -1; ret = -1;
goto out;
} }
nd = &nd_table[idx]; nd = &nd_table[idx];
macaddr = nd->macaddr; macaddr = nd->macaddr;
...@@ -1624,7 +1625,8 @@ int net_client_init(const char *device, const char *p) ...@@ -1624,7 +1625,8 @@ int net_client_init(const char *device, const char *p)
if (get_param_value(buf, sizeof(buf), "macaddr", p)) { if (get_param_value(buf, sizeof(buf), "macaddr", p)) {
if (parse_macaddr(macaddr, buf) < 0) { if (parse_macaddr(macaddr, buf) < 0) {
fprintf(stderr, "invalid syntax for ethernet address\n"); fprintf(stderr, "invalid syntax for ethernet address\n");
return -1; ret = -1;
goto out;
} }
} }
if (get_param_value(buf, sizeof(buf), "model", p)) { if (get_param_value(buf, sizeof(buf), "model", p)) {
...@@ -1664,8 +1666,9 @@ int net_client_init(const char *device, const char *p) ...@@ -1664,8 +1666,9 @@ int net_client_init(const char *device, const char *p)
port = strtol(p, &devname, 10); port = strtol(p, &devname, 10);
devname++; devname++;
if (port < 1 || port > 65535) { if (port < 1 || port > 65535) {
fprintf(stderr, "vmchannel wrong port number\n"); fprintf(stderr, "vmchannel wrong port number\n");
return -1; ret = -1;
goto out;
} }
vmc = malloc(sizeof(struct VMChannel)); vmc = malloc(sizeof(struct VMChannel));
snprintf(name, 20, "vmchannel%ld", port); snprintf(name, 20, "vmchannel%ld", port);
...@@ -1673,7 +1676,8 @@ int net_client_init(const char *device, const char *p) ...@@ -1673,7 +1676,8 @@ int net_client_init(const char *device, const char *p)
if (!vmc->hd) { if (!vmc->hd) {
fprintf(stderr, "qemu: could not open vmchannel device" fprintf(stderr, "qemu: could not open vmchannel device"
"'%s'\n", devname); "'%s'\n", devname);
return -1; ret = -1;
goto out;
} }
vmc->port = port; vmc->port = port;
slirp_add_exec(3, vmc->hd, 4, port); slirp_add_exec(3, vmc->hd, 4, port);
...@@ -1687,7 +1691,8 @@ int net_client_init(const char *device, const char *p) ...@@ -1687,7 +1691,8 @@ int net_client_init(const char *device, const char *p)
char ifname[64]; char ifname[64];
if (get_param_value(ifname, sizeof(ifname), "ifname", p) <= 0) { if (get_param_value(ifname, sizeof(ifname), "ifname", p) <= 0) {
fprintf(stderr, "tap: no interface name\n"); fprintf(stderr, "tap: no interface name\n");
return -1; ret = -1;
goto out;
} }
vlan->nb_host_devs++; vlan->nb_host_devs++;
ret = tap_win32_init(vlan, device, name, ifname); ret = tap_win32_init(vlan, device, name, ifname);
...@@ -1734,7 +1739,8 @@ int net_client_init(const char *device, const char *p) ...@@ -1734,7 +1739,8 @@ int net_client_init(const char *device, const char *p)
ret = net_socket_mcast_init(vlan, device, name, buf); ret = net_socket_mcast_init(vlan, device, name, buf);
} else { } else {
fprintf(stderr, "Unknown socket options: %s\n", p); fprintf(stderr, "Unknown socket options: %s\n", p);
return -1; ret = -1;
goto out;
} }
vlan->nb_host_devs++; vlan->nb_host_devs++;
} else } else
...@@ -1764,13 +1770,13 @@ int net_client_init(const char *device, const char *p) ...@@ -1764,13 +1770,13 @@ int net_client_init(const char *device, const char *p)
#endif #endif
{ {
fprintf(stderr, "Unknown network device: %s\n", device); fprintf(stderr, "Unknown network device: %s\n", device);
if (name) ret = -1;
free(name); goto out;
return -1;
} }
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "Could not initialize device '%s'\n", device); fprintf(stderr, "Could not initialize device '%s'\n", device);
} }
out:
if (name) if (name)
free(name); free(name);
return ret; return ret;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册