[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/18] slirp: goto bad in udp_input if sosendto
From: |
Wen Congyang |
Subject: |
Re: [Qemu-devel] [PATCH 01/18] slirp: goto bad in udp_input if sosendto fails |
Date: |
Wed, 29 Jul 2015 13:32:46 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 |
On 07/29/2015 06:57 AM, Samuel Thibault wrote:
> Before this patch, if sosendto fails, udp_input is executed as if the
> packet was sent, recording the packet for icmp errors, which does not
> makes sense since the packet was not actually sent, errors would be
> related to a previous packet.
>
> This patch adds a goto bad to cut the execution of this function.
>
> Signed-off-by: Guillaume Subiron <address@hidden>
> ---
> slirp/debug.h | 2 +-
> slirp/udp.c | 1 +
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/slirp/debug.h b/slirp/debug.h
> index 6cfa61e..c60f967 100644
> --- a/slirp/debug.h
> +++ b/slirp/debug.h
> @@ -5,7 +5,7 @@
> * terms and conditions of the copyright.
> */
>
> -//#define DEBUG 1
> +#define DEBUG 1
Why change this line?
Thanks
Wen Congyang
>
> #ifdef DEBUG
>
> diff --git a/slirp/udp.c b/slirp/udp.c
> index f77e00f..ed78274 100644
> --- a/slirp/udp.c
> +++ b/slirp/udp.c
> @@ -218,6 +218,7 @@ udp_input(register struct mbuf *m, int iphlen)
> *ip=save_ip;
> DEBUG_MISC((dfd,"udp tx errno = %d-%s\n",errno,strerror(errno)));
> icmp_error(m, ICMP_UNREACH,ICMP_UNREACH_NET, 0,strerror(errno));
> + goto bad;
> }
>
> m_free(so->so_m); /* used for ICMP if error on sorecvfrom */
>
- [Qemu-devel] [PATCH 12/18] slirp: Adding IPv6 UDP support, (continued)
- [Qemu-devel] [PATCH 12/18] slirp: Adding IPv6 UDP support, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 17/18] slirp: Adding IPv6 address for DNS relay, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 16/18] slirp: Handle IPv6 in TCP functions, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 15/18] slirp: Reindent after refactoring, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 13/18] slirp: Factorizing tcpiphdr structure with an union, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 11/18] slirp: Adding ICMPv6 error sending, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 14/18] slirp: Generalizing and neutralizing various TCP functions before adding IPv6 stuff, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 18/18] qapi-schema, qemu-options & slirp: Adding Qemu options for IPv6 addresses, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 05/18] slirp: Factorizing address translation, Samuel Thibault, 2015/07/28
- [Qemu-devel] [PATCH 10/18] slirp: Adding IPv6, ICMPv6 Echo and NDP autoconfiguration, Samuel Thibault, 2015/07/28
- Re: [Qemu-devel] [PATCH 01/18] slirp: goto bad in udp_input if sosendto fails,
Wen Congyang <=
- Re: [Qemu-devel] [PATCHv4 00/18] slirp: Adding IPv6 support to Qemu -net user mode, Jan Kiszka, 2015/07/29