diff options
| author | Alexey Kirillov <lekiravi@yandex-team.ru> | 2021-03-03 12:59:08 +0300 |
|---|---|---|
| committer | Jason Wang <jasowang@redhat.com> | 2021-03-15 16:41:22 +0800 |
| commit | 59b5437eb732d6b103a9bc279c3482c834d1eff9 (patch) | |
| tree | 9f0667074f09bf4fe8f7ac904a030e72189d9774 /net/slirp.c | |
| parent | 3c3b656885473ef0d699290ba966177f17839aa5 (diff) | |
| download | focaccia-qemu-59b5437eb732d6b103a9bc279c3482c834d1eff9.tar.gz focaccia-qemu-59b5437eb732d6b103a9bc279c3482c834d1eff9.zip | |
net: Move NetClientState.info_str to dynamic allocations
The info_str field of the NetClientState structure is static and has a size of 256 bytes. This amount is often unclaimed, and the field itself is used exclusively for HMP "info network". The patch translates info_str to dynamic memory allocation. This action is also allows us to painlessly discard usage of this field for backend devices. Signed-off-by: Alexey Kirillov <lekiravi@yandex-team.ru> Signed-off-by: Jason Wang <jasowang@redhat.com>
Diffstat (limited to 'net/slirp.c')
| -rw-r--r-- | net/slirp.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/slirp.c b/net/slirp.c index 6ab348b943..bfa07e3432 100644 --- a/net/slirp.c +++ b/net/slirp.c @@ -664,9 +664,8 @@ static int net_slirp_init(NetClientState *peer, const char *model, stored->tftp_server_name = g_strdup(tftp_server_name); } - snprintf(nc->info_str, sizeof(nc->info_str), - "net=%s,restrict=%s", inet_ntoa(net), - restricted ? "on" : "off"); + nc->info_str = g_strdup_printf("net=%s,restrict=%s", inet_ntoa(net), + restricted ? "on" : "off"); s = DO_UPCAST(SlirpState, nc, nc); |