diff options
| author | Patrick McHardy <kaber@trash.net> | 2007-12-05 01:30:29 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-01-28 14:56:22 -0800 |
| commit | 861934c7c888973da8bf621b3959e408531539e1 (patch) | |
| tree | dd0f0c008e58c68bdb35129b00657ac873944461 /net | |
| parent | ea3a66ff5ae41daa1d1aa789140a70415eaf7b46 (diff) | |
[NETFILTER]: nfnetlink_queue: eliminate impossible switch case
We don't need a default case in nfqnl_build_packet_message(), the
copy_mode is validated when it is set. Tell the compiler about
the possible types and remove the default case. Saves 80b of
text on x86_64.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/netfilter/nfnetlink_queue.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/net/netfilter/nfnetlink_queue.c b/net/netfilter/nfnetlink_queue.c index 436b442d4ed..abd5ff9b89a 100644 --- a/net/netfilter/nfnetlink_queue.c +++ b/net/netfilter/nfnetlink_queue.c @@ -232,7 +232,7 @@ nfqnl_build_packet_message(struct nfqnl_instance *queue, spin_lock_bh(&queue->lock); - switch (queue->copy_mode) { + switch ((enum nfqnl_config_mode)queue->copy_mode) { case NFQNL_COPY_META: case NFQNL_COPY_NONE: data_len = 0; @@ -253,11 +253,6 @@ nfqnl_build_packet_message(struct nfqnl_instance *queue, size += nla_total_size(data_len); break; - - default: - *errp = -EINVAL; - spin_unlock_bh(&queue->lock); - return NULL; } entry->id = queue->id_sequence++; |