qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH qemu] slirp/debug: Print IP addre


From: Paolo Bonzini
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH qemu] slirp/debug: Print IP addresses in human readable form
Date: Wed, 7 Mar 2018 15:47:41 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

On 01/02/2018 10:35, Alexey Kardashevskiy wrote:
> Signed-off-by: Alexey Kardashevskiy <address@hidden>
> ---
>  slirp/arp_table.c | 4 ++--
>  slirp/socket.c    | 8 ++++----
>  slirp/udp.c       | 4 ++--
>  3 files changed, 8 insertions(+), 8 deletions(-)

Samuel, Jason, can you pick this up?

Thanks,

Paolo

> diff --git a/slirp/arp_table.c b/slirp/arp_table.c
> index 3547043..bac608f 100644
> --- a/slirp/arp_table.c
> +++ b/slirp/arp_table.c
> @@ -33,7 +33,7 @@ void arp_table_add(Slirp *slirp, uint32_t ip_addr, uint8_t 
> ethaddr[ETH_ALEN])
>      int i;
>  
>      DEBUG_CALL("arp_table_add");
> -    DEBUG_ARG("ip = 0x%x", ip_addr);
> +    DEBUG_ARG("ip = %s", inet_ntoa(*(struct in_addr *)&ip_addr));
>      DEBUG_ARGS((dfd, " hw addr = %02x:%02x:%02x:%02x:%02x:%02x\n",
>                  ethaddr[0], ethaddr[1], ethaddr[2],
>                  ethaddr[3], ethaddr[4], ethaddr[5]));
> @@ -67,7 +67,7 @@ bool arp_table_search(Slirp *slirp, uint32_t ip_addr,
>      int i;
>  
>      DEBUG_CALL("arp_table_search");
> -    DEBUG_ARG("ip = 0x%x", ip_addr);
> +    DEBUG_ARG("ip = %s", inet_ntoa(*(struct in_addr *)&ip_addr));
>  
>      /* If broadcast address */
>      if (ip_addr == 0xffffffff || ip_addr == broadcast_addr) {
> diff --git a/slirp/socket.c b/slirp/socket.c
> index cb7b5b6..61347d1 100644
> --- a/slirp/socket.c
> +++ b/slirp/socket.c
> @@ -701,10 +701,10 @@ tcp_listen(Slirp *slirp, uint32_t haddr, u_int hport, 
> uint32_t laddr,
>       memset(&addr, 0, addrlen);
>  
>       DEBUG_CALL("tcp_listen");
> -     DEBUG_ARG("haddr = %x", haddr);
> -     DEBUG_ARG("hport = %d", hport);
> -     DEBUG_ARG("laddr = %x", laddr);
> -     DEBUG_ARG("lport = %d", lport);
> +     DEBUG_ARG("haddr = %s", inet_ntoa(*(struct in_addr *)&haddr));
> +     DEBUG_ARG("hport = %d", ntohs(hport));
> +     DEBUG_ARG("laddr = %s", inet_ntoa(*(struct in_addr *)&laddr));
> +     DEBUG_ARG("lport = %d", ntohs(lport));
>       DEBUG_ARG("flags = %x", flags);
>  
>       so = socreate(slirp);
> diff --git a/slirp/udp.c b/slirp/udp.c
> index 227d779..e5bf065 100644
> --- a/slirp/udp.c
> +++ b/slirp/udp.c
> @@ -241,8 +241,8 @@ int udp_output(struct socket *so, struct mbuf *m,
>       DEBUG_CALL("udp_output");
>       DEBUG_ARG("so = %p", so);
>       DEBUG_ARG("m = %p", m);
> -     DEBUG_ARG("saddr = %lx", (long)saddr->sin_addr.s_addr);
> -     DEBUG_ARG("daddr = %lx", (long)daddr->sin_addr.s_addr);
> +     DEBUG_ARG("saddr = %s", inet_ntoa(saddr->sin_addr));
> +     DEBUG_ARG("daddr = %s", inet_ntoa(daddr->sin_addr));
>  
>       /*
>        * Adjust for header
> 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]