Просмотр исходного кода

[NET]: Remove redundant code in net/core/filter.c

skb_header_pointer handles linear and non-linear data, no need to handle
linear data again.

Signed-off-by: Patrick McHardy <kaber@trash.net>
Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Patrick McHardy 20 лет назад
Родитель
Сommit
6935d46c2d
1 измененных файлов с 0 добавлено и 12 удалено
  1. 0 12
      net/core/filter.c

+ 0 - 12
net/core/filter.c

@@ -168,10 +168,6 @@ int sk_run_filter(struct sk_buff *skb, struct sock_filter *filter, int flen)
 		case BPF_LD|BPF_W|BPF_ABS:
 		case BPF_LD|BPF_W|BPF_ABS:
 			k = fentry->k;
 			k = fentry->k;
  load_w:
  load_w:
-			if (k >= 0 && (unsigned int)(k+sizeof(u32)) <= len) {
-				A = ntohl(*(u32*)&data[k]);
-				continue;
-			}
 			if (k < 0) {
 			if (k < 0) {
 				u8 *ptr;
 				u8 *ptr;
 
 
@@ -194,10 +190,6 @@ int sk_run_filter(struct sk_buff *skb, struct sock_filter *filter, int flen)
 		case BPF_LD|BPF_H|BPF_ABS:
 		case BPF_LD|BPF_H|BPF_ABS:
 			k = fentry->k;
 			k = fentry->k;
  load_h:
  load_h:
-			if (k >= 0 && (unsigned int)(k + sizeof(u16)) <= len) {
-				A = ntohs(*(u16*)&data[k]);
-				continue;
-			}
 			if (k < 0) {
 			if (k < 0) {
 				u8 *ptr;
 				u8 *ptr;
 
 
@@ -220,10 +212,6 @@ int sk_run_filter(struct sk_buff *skb, struct sock_filter *filter, int flen)
 		case BPF_LD|BPF_B|BPF_ABS:
 		case BPF_LD|BPF_B|BPF_ABS:
 			k = fentry->k;
 			k = fentry->k;
 load_b:
 load_b:
-			if (k >= 0 && (unsigned int)k < len) {
-				A = data[k];
-				continue;
-			}
 			if (k < 0) {
 			if (k < 0) {
 				u8 *ptr;
 				u8 *ptr;