mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-23 09:19:51 +00:00
[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>
This commit is contained in:
parent
ea3a66ff5a
commit
861934c7c8
1 changed files with 1 additions and 6 deletions
|
@ -232,7 +232,7 @@ nfqnl_build_packet_message(struct nfqnl_instance *queue,
|
||||||
|
|
||||||
spin_lock_bh(&queue->lock);
|
spin_lock_bh(&queue->lock);
|
||||||
|
|
||||||
switch (queue->copy_mode) {
|
switch ((enum nfqnl_config_mode)queue->copy_mode) {
|
||||||
case NFQNL_COPY_META:
|
case NFQNL_COPY_META:
|
||||||
case NFQNL_COPY_NONE:
|
case NFQNL_COPY_NONE:
|
||||||
data_len = 0;
|
data_len = 0;
|
||||||
|
@ -253,11 +253,6 @@ nfqnl_build_packet_message(struct nfqnl_instance *queue,
|
||||||
|
|
||||||
size += nla_total_size(data_len);
|
size += nla_total_size(data_len);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
default:
|
|
||||||
*errp = -EINVAL;
|
|
||||||
spin_unlock_bh(&queue->lock);
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
entry->id = queue->id_sequence++;
|
entry->id = queue->id_sequence++;
|
||||||
|
|
Loading…
Reference in a new issue