Re: [PATCH 4/6] PKT_SCHED: u32 ematch

[Thomas Graf <tgraf@suug.ch>]

* Patrick McHardy <41F4402D.6040200@trash.net> 2005-01-24 01:24
> Thomas Graf wrote:
> >+static int em_u32_match(struct sk_buff *skb, struct tcf_ematch *em,
> >+			struct tcf_pkt_info *info)
> >+{
> >+	struct tc_u32_key *key = (struct tc_u32_key *) em->data;
> >+	unsigned char *ptr = skb->nh.raw;
> >+	
> >+	if (info) {
> >+		if (info->ptr)
> >+			ptr = info->ptr;
> >+		ptr += (info->nexthdr & key->offmask);
> >+	}
> >+	
> >+	return !((*(u32*) (ptr + key->off) ^ key->val) & key->mask);
> > 
> >
> This needs to make sure it stays inside the skb. I know cls_u32's checks
> are pretty weak, but having nothing doesn't seem right.

It's a exact copy with all the weaknesses inherited. I'll add a
call to tcf_valid_offset, we'll lose the _feature_ to match areas
in the page beyond the skb data though. ;->