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/netlink/genetlink.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/netlink/genetlink.c')
| -rw-r--r-- | net/netlink/genetlink.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/netlink/genetlink.c b/net/netlink/genetlink.c index d16929c9b4bc..f5aa23c3e886 100644 --- a/net/netlink/genetlink.c +++ b/net/netlink/genetlink.c @@ -554,7 +554,8 @@ static int ctrl_fill_info(struct genl_family *family, u32 pid, u32 seq, return genlmsg_end(skb, hdr); nla_put_failure: - return genlmsg_cancel(skb, hdr); + genlmsg_cancel(skb, hdr); + return -EMSGSIZE; } static int ctrl_fill_mcgrp_info(struct genl_multicast_group *grp, u32 pid, @@ -590,7 +591,8 @@ static int ctrl_fill_mcgrp_info(struct genl_multicast_group *grp, u32 pid, return genlmsg_end(skb, hdr); nla_put_failure: - return genlmsg_cancel(skb, hdr); + genlmsg_cancel(skb, hdr); + return -EMSGSIZE; } static int ctrl_dumpfamily(struct sk_buff *skb, struct netlink_callback *cb) |