[Qemu-devel] [PATCH v2 0/4] slirp: Fix for requeuing crash, cleanups

[Jan Kiszka]

Well, this requeuing bug seems to have a long breath. Previous attempts
to fix it (mine included) neglected the fact that we need to walk the
queue of pending packets, not just restart from the beginning after a
requeue. This version should get it Right(TM).

This also comes with a fix for resource cleanups on slirp shutdown. At
least valgrind is happy now.

Changes in v2:
 - fixed corner case of session list walk that Stefan Weil reported

CC: Fabien Chouteau <address@hidden>
CC: Michael S. Tsirkin <address@hidden>
CC: Stefan Weil <address@hidden>
CC: Zhi Yong Wu <address@hidden>

Jan Kiszka (4):
  slirp: Keep next_m always valid
  slirp: Fix queue walking in if_start
  slirp: Remove unneeded if_queued
  slirp: Cleanup resources on instance removal

 slirp/if.c       |   64 +++++++++++++++++++++++++++++------------------------
 slirp/ip_icmp.c  |    7 ++++++
 slirp/ip_icmp.h  |    1 +
 slirp/ip_input.c |    7 ++++++
 slirp/mbuf.c     |   21 +++++++++++++++++
 slirp/mbuf.h     |    1 +
 slirp/slirp.c    |   10 +++-----
 slirp/slirp.h    |    3 +-
 slirp/tcp_subr.c |    7 ++++++
 slirp/udp.c      |    8 ++++++
 slirp/udp.h      |    1 +
 11 files changed, 94 insertions(+), 36 deletions(-)

-- 
1.7.3.4