diff options
| author | Thomas Graf <tgraf@suug.ch> | 2008-06-04 01:36:54 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-06-04 01:36:54 +0200 |
| commit | bc3ed28caaef55e7e3a9316464256353c5f9b1df (patch) | |
| tree | 3aed4521aa2d74a36ee2b192c2e229fd23fbe732 /net/sched/sch_dsmark.c | |
| parent | route: Mark unused routing attributes as such (diff) | |
| download | linux-bc3ed28caaef55e7e3a9316464256353c5f9b1df.tar.xz linux-bc3ed28caaef55e7e3a9316464256353c5f9b1df.zip | |
netlink: Improve returned error codes
Make nlmsg_trim(), nlmsg_cancel(), genlmsg_cancel(), and
nla_nest_cancel() void functions.
Return -EMSGSIZE instead of -1 if the provided message buffer is not
big enough.
Signed-off-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/sch_dsmark.c')
| -rw-r--r-- | net/sched/sch_dsmark.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/sched/sch_dsmark.c b/net/sched/sch_dsmark.c index 0df911fd67b1..64465bacbe79 100644 --- a/net/sched/sch_dsmark.c +++ b/net/sched/sch_dsmark.c @@ -444,7 +444,8 @@ static int dsmark_dump_class(struct Qdisc *sch, unsigned long cl, return nla_nest_end(skb, opts); nla_put_failure: - return nla_nest_cancel(skb, opts); + nla_nest_cancel(skb, opts); + return -EMSGSIZE; } static int dsmark_dump(struct Qdisc *sch, struct sk_buff *skb) @@ -466,7 +467,8 @@ static int dsmark_dump(struct Qdisc *sch, struct sk_buff *skb) return nla_nest_end(skb, opts); nla_put_failure: - return nla_nest_cancel(skb, opts); + nla_nest_cancel(skb, opts); + return -EMSGSIZE; } static const struct Qdisc_class_ops dsmark_class_ops = { |