[PKT_SCHED]: Fix error handling while dumping actions
authorThomas Graf <tgraf@suug.ch>
Thu, 6 Jul 2006 03:47:28 +0000 (20:47 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 6 Jul 2006 03:47:28 +0000 (20:47 -0700)
"return -err" and blindly inheriting the error code in the netlink
failure exception handler causes errors codes to be returned as
positive value therefore making them being ignored by the caller.

May lead to sending out incomplete netlink messages.

Signed-off-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/act_api.c

index 9b2e3975be0bbdb32303e29f57dab34ab8b01983..599423cc9d0d67796c4936f1d15c08fe979d1e0e 100644 (file)
@@ -250,15 +250,17 @@ tcf_action_dump(struct sk_buff *skb, struct tc_action *act, int bind, int ref)
                RTA_PUT(skb, a->order, 0, NULL);
                err = tcf_action_dump_1(skb, a, bind, ref);
                if (err < 0)
-                       goto rtattr_failure;
+                       goto errout;
                r->rta_len = skb->tail - (u8*)r;
        }
 
        return 0;
 
 rtattr_failure:
+       err = -EINVAL;
+errout:
        skb_trim(skb, b - skb->data);
-       return -err;
+       return err;
 }
 
 struct tc_action *tcf_action_init_1(struct rtattr *rta, struct rtattr *est,