aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhilippe Mathieu-Daudé <philmd@redhat.com>2020-03-07 15:13:42 +0100
committerMarc-André Lureau <marcandre.lureau@redhat.com>2020-07-20 00:21:40 +0400
commit79c3b0c2d9d97b2ae818a70eabe4dbc2696f95b8 (patch)
tree8ee37f61dac0e462a702db9f421955c7ab67683e
parent2bbd06092ca3f0427dab1536374f5824e640767c (diff)
downloadslirp-79c3b0c2d9d97b2ae818a70eabe4dbc2696f95b8.zip
slirp-79c3b0c2d9d97b2ae818a70eabe4dbc2696f95b8.tar.gz
slirp-79c3b0c2d9d97b2ae818a70eabe4dbc2696f95b8.tar.bz2
Remove unnecessary break
The code is unreachable, so no need to break. This silence static analyzer warnings. Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
-rw-r--r--src/ip6_icmp.c1
-rw-r--r--src/slirp.c1
-rw-r--r--src/socket.c3
3 files changed, 0 insertions, 5 deletions
diff --git a/src/ip6_icmp.c b/src/ip6_icmp.c
index 28ec2be..d9c872b 100644
--- a/src/ip6_icmp.c
+++ b/src/ip6_icmp.c
@@ -119,7 +119,6 @@ void icmp6_send_error(struct mbuf *m, uint8_t type, uint8_t code)
break;
default:
g_assert_not_reached();
- break;
}
t->m_data += ICMP6_ERROR_MINLEN;
memcpy(t->m_data, m->m_data, error_data_len);
diff --git a/src/slirp.c b/src/slirp.c
index 7effcf5..dba7c98 100644
--- a/src/slirp.c
+++ b/src/slirp.c
@@ -968,7 +968,6 @@ int if_encap(Slirp *slirp, struct mbuf *ifm)
default:
g_assert_not_reached();
- break;
}
memcpy(eh->h_dest, ethaddr, ETH_ALEN);
diff --git a/src/socket.c b/src/socket.c
index 4cd9a64..1e385df 100644
--- a/src/socket.c
+++ b/src/socket.c
@@ -551,7 +551,6 @@ void sorecvfrom(struct socket *so)
break;
default:
g_assert_not_reached();
- break;
}
/*
@@ -603,7 +602,6 @@ void sorecvfrom(struct socket *so)
break;
default:
g_assert_not_reached();
- break;
}
m_free(m);
} else {
@@ -639,7 +637,6 @@ void sorecvfrom(struct socket *so)
break;
default:
g_assert_not_reached();
- break;
}
} /* rx error */
} /* if ping packet */