diff options
author | Daniel Borkmann <dxchgb@gmail.com> | 2012-10-24 09:27:24 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-10-26 02:17:20 -0400 |
commit | 342567ccf088a6c4777ef352f559fe46aaeb047e (patch) | |
tree | f6093fee9ed4fabf34697c63693f04d951b977cf /net/packet | |
parent | e0c075577965d1c01b30038d38bf637b027a1df3 (diff) |
packet: minor: remove unused err assignment
This tiny patch removes two unused err assignments. In those two cases the
err variable is either overwritten with another value at a later point in
time without having read the previous assigment, or it is assigned and the
function returns without using/reading err after the assignment.
Signed-off-by: Daniel Borkmann <daniel.borkmann@tik.ee.ethz.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/packet')
-rw-r--r-- | net/packet/af_packet.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 94060edbbd70..9034f52659b5 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c | |||
@@ -1907,7 +1907,6 @@ static int tpacket_fill_skb(struct packet_sock *po, struct sk_buff *skb, | |||
1907 | to_write -= dev->hard_header_len; | 1907 | to_write -= dev->hard_header_len; |
1908 | } | 1908 | } |
1909 | 1909 | ||
1910 | err = -EFAULT; | ||
1911 | offset = offset_in_page(data); | 1910 | offset = offset_in_page(data); |
1912 | len_max = PAGE_SIZE - offset; | 1911 | len_max = PAGE_SIZE - offset; |
1913 | len = ((to_write > len_max) ? len_max : to_write); | 1912 | len = ((to_write > len_max) ? len_max : to_write); |
@@ -1957,7 +1956,6 @@ static int tpacket_snd(struct packet_sock *po, struct msghdr *msg) | |||
1957 | 1956 | ||
1958 | mutex_lock(&po->pg_vec_lock); | 1957 | mutex_lock(&po->pg_vec_lock); |
1959 | 1958 | ||
1960 | err = -EBUSY; | ||
1961 | if (saddr == NULL) { | 1959 | if (saddr == NULL) { |
1962 | dev = po->prot_hook.dev; | 1960 | dev = po->prot_hook.dev; |
1963 | proto = po->num; | 1961 | proto = po->num; |