aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick McHardy <kaber@trash.net>2007-06-25 17:30:16 -0400
committerDavid S. Miller <davem@sunset.davemloft.net>2007-07-11 01:15:39 -0400
commitafdc3238ec948531205f5c5f77d2de7bae519c71 (patch)
treece69a6ed3095ba5049673340304672381068b3af
parent1092cb219774a82b1f16781aec7b8d4ec727c981 (diff)
[RTNETLINK]: Add nested compat attribute
Add a nested compat attribute type that can be used to convert attributes that contain a structure to nested attributes in a backwards compatible way. The attribute looks like this: struct { [ compat contents ] struct rtattr { .rta_len = total size, .rta_type = type, } rta; struct old_structure struct; [ nested top-level attribute ] struct rtattr { .rta_len = nest size, .rta_type = type, } nest_attr; [ optional 0 .. n nested attributes ] struct rtattr { .rta_len = private attribute len, .rta_type = private attribute typ, } nested_attr; struct nested_data data; }; Since both userspace and kernel deal correctly with attributes that are larger than expected old versions will just parse the compat part and ignore the rest. Signed-off-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--include/linux/rtnetlink.h14
-rw-r--r--net/core/rtnetlink.c16
2 files changed, 30 insertions, 0 deletions
diff --git a/include/linux/rtnetlink.h b/include/linux/rtnetlink.h
index 612785848532..6731e7f4cc0f 100644
--- a/include/linux/rtnetlink.h
+++ b/include/linux/rtnetlink.h
@@ -570,6 +570,8 @@ static __inline__ int rtattr_strcmp(const struct rtattr *rta, const char *str)
570} 570}
571 571
572extern int rtattr_parse(struct rtattr *tb[], int maxattr, struct rtattr *rta, int len); 572extern int rtattr_parse(struct rtattr *tb[], int maxattr, struct rtattr *rta, int len);
573extern int rtattr_parse_nested_compat(struct rtattr *tb[], int maxattr,
574 struct rtattr *rta, void **data, int len);
573 575
574#define rtattr_parse_nested(tb, max, rta) \ 576#define rtattr_parse_nested(tb, max, rta) \
575 rtattr_parse((tb), (max), RTA_DATA((rta)), RTA_PAYLOAD((rta))) 577 rtattr_parse((tb), (max), RTA_DATA((rta)), RTA_PAYLOAD((rta)))
@@ -638,6 +640,18 @@ extern void __rta_fill(struct sk_buff *skb, int attrtype, int attrlen, const voi
638({ (start)->rta_len = skb_tail_pointer(skb) - (unsigned char *)(start); \ 640({ (start)->rta_len = skb_tail_pointer(skb) - (unsigned char *)(start); \
639 (skb)->len; }) 641 (skb)->len; })
640 642
643#define RTA_NEST_COMPAT(skb, type, attrlen, data) \
644({ struct rtattr *__start = (struct rtattr *)skb_tail_pointer(skb); \
645 RTA_PUT(skb, type, attrlen, data); \
646 RTA_NEST(skb, type); \
647 __start; })
648
649#define RTA_NEST_COMPAT_END(skb, start) \
650({ struct rtattr *__nest = (void *)(start) + NLMSG_ALIGN((start)->rta_len); \
651 (start)->rta_len = skb_tail_pointer(skb) - (unsigned char *)(start); \
652 RTA_NEST_END(skb, __nest); \
653 (skb)->len; })
654
641#define RTA_NEST_CANCEL(skb, start) \ 655#define RTA_NEST_CANCEL(skb, start) \
642({ if (start) \ 656({ if (start) \
643 skb_trim(skb, (unsigned char *) (start) - (skb)->data); \ 657 skb_trim(skb, (unsigned char *) (start) - (skb)->data); \
diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
index 06c0c5afabf0..c25d23ba6d5d 100644
--- a/net/core/rtnetlink.c
+++ b/net/core/rtnetlink.c
@@ -97,6 +97,21 @@ int rtattr_parse(struct rtattr *tb[], int maxattr, struct rtattr *rta, int len)
97 return 0; 97 return 0;
98} 98}
99 99
100int rtattr_parse_nested_compat(struct rtattr *tb[], int maxattr,
101 struct rtattr *rta, void **data, int len)
102{
103 if (RTA_PAYLOAD(rta) < len)
104 return -1;
105 *data = RTA_DATA(rta);
106
107 if (RTA_PAYLOAD(rta) >= RTA_ALIGN(len) + sizeof(struct rtattr)) {
108 rta = RTA_DATA(rta) + RTA_ALIGN(len);
109 return rtattr_parse_nested(tb, maxattr, rta);
110 }
111 memset(tb, 0, sizeof(struct rtattr *) * maxattr);
112 return 0;
113}
114
100static struct rtnl_link *rtnl_msg_handlers[NPROTO]; 115static struct rtnl_link *rtnl_msg_handlers[NPROTO];
101 116
102static inline int rtm_msgindex(int msgtype) 117static inline int rtm_msgindex(int msgtype)
@@ -1297,6 +1312,7 @@ void __init rtnetlink_init(void)
1297EXPORT_SYMBOL(__rta_fill); 1312EXPORT_SYMBOL(__rta_fill);
1298EXPORT_SYMBOL(rtattr_strlcpy); 1313EXPORT_SYMBOL(rtattr_strlcpy);
1299EXPORT_SYMBOL(rtattr_parse); 1314EXPORT_SYMBOL(rtattr_parse);
1315EXPORT_SYMBOL(rtattr_parse_nested_compat);
1300EXPORT_SYMBOL(rtnetlink_put_metrics); 1316EXPORT_SYMBOL(rtnetlink_put_metrics);
1301EXPORT_SYMBOL(rtnl_lock); 1317EXPORT_SYMBOL(rtnl_lock);
1302EXPORT_SYMBOL(rtnl_trylock); 1318EXPORT_SYMBOL(rtnl_trylock);