diff options
author | Hani Benhabiles <kroosec@gmail.com> | 2014-01-09 19:34:27 +0100 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@redhat.com> | 2014-01-27 15:44:06 +0100 |
commit | 4bf2c138ddefc6ff17f6c4b947320c60aa0c38a0 (patch) | |
tree | 92720482ef1f706c07efc2c1f031cfbd5a87847c | |
parent | 0169c511554cb0014a00290b0d3d26c31a49818f (diff) | |
download | qemu-4bf2c138ddefc6ff17f6c4b947320c60aa0c38a0.zip |
net: Use g_strdup_printf instead of snprintf.
assign_name() in net/net.c is using snprintf + g_strdup to get the same
result as g_strdup_printf.
Signed-off-by: Hani Benhabiles <kroosec@gmail.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
-rw-r--r-- | net/net.c | 5 |
1 files changed, 1 insertions, 4 deletions
@@ -164,7 +164,6 @@ void qemu_macaddr_default_if_unset(MACAddr *macaddr) static char *assign_name(NetClientState *nc1, const char *model) { NetClientState *nc; - char buf[256]; int id = 0; QTAILQ_FOREACH(nc, &net_clients, next) { @@ -176,9 +175,7 @@ static char *assign_name(NetClientState *nc1, const char *model) } } - snprintf(buf, sizeof(buf), "%s.%d", model, id); - - return g_strdup(buf); + return g_strdup_printf("%s.%d", model, id); } static void qemu_net_client_destructor(NetClientState *nc) |