[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]



I was looking through the source to the .88 kernel and ran across this:

static int audit_receive_skb(struct sk_buff *skb)
        int             err;
        struct nlmsghdr *nlh;
        u32             rlen;

        while (skb->len >= NLMSG_SPACE(0)) {
                nlh = (struct nlmsghdr *)skb->data;
                if (nlh->nlmsg_len < sizeof(*nlh) || skb->len < 
                        return 0;
                rlen = NLMSG_ALIGN(nlh->nlmsg_len);
                if (rlen > skb->len)
                        rlen = skb->len;
                if ((err = audit_receive_msg(skb, nlh))) {
                        netlink_ack(skb, nlh, err);
                } else if (nlh->nlmsg_flags & NLM_F_ACK)
                        netlink_ack(skb, nlh, 0);
                skb_pull(skb, rlen);
        return 0;

It only returns 0. Is this a mistake or should this be made void? The reason I 
ask is that the return code is used like this:

                if (audit_receive_skb(skb) && skb->len)
                        skb_queue_head(&sk->sk_receive_queue, skb);

The way the code is, we will never put the skb back on the queue head. Should 
this be refactored or do we have a problem in the .88 kernel?


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]