diff options
author | Willem de Bruijn <willemb@google.com> | 2016-03-09 21:58:33 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-03-09 22:13:01 -0500 |
commit | ea47781c26510e5d97f80f9aceafe9065bd5e3aa (patch) | |
tree | 155f27a643ca906b31d66a6cff3440fdea97b5cb | |
parent | 2793a23aacbd754dbbb5cb75093deb7e4103bace (diff) |
ax25: add link layer header validation function
As variable length protocol, AX25 fails link layer header validation
tests based on a minimum length. header_ops.validate allows protocols
to validate headers that are shorter than hard_header_len. Implement
this callback for AX25.
See also http://comments.gmane.org/gmane.linux.network/401064
Signed-off-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ax25/ax25_ip.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/net/ax25/ax25_ip.c b/net/ax25/ax25_ip.c index b563a3f5f2a8..2fa3be965101 100644 --- a/net/ax25/ax25_ip.c +++ b/net/ax25/ax25_ip.c | |||
@@ -228,8 +228,23 @@ netdev_tx_t ax25_ip_xmit(struct sk_buff *skb) | |||
228 | } | 228 | } |
229 | #endif | 229 | #endif |
230 | 230 | ||
231 | static bool ax25_validate_header(const char *header, unsigned int len) | ||
232 | { | ||
233 | ax25_digi digi; | ||
234 | |||
235 | if (!len) | ||
236 | return false; | ||
237 | |||
238 | if (header[0]) | ||
239 | return true; | ||
240 | |||
241 | return ax25_addr_parse(header + 1, len - 1, NULL, NULL, &digi, NULL, | ||
242 | NULL); | ||
243 | } | ||
244 | |||
231 | const struct header_ops ax25_header_ops = { | 245 | const struct header_ops ax25_header_ops = { |
232 | .create = ax25_hard_header, | 246 | .create = ax25_hard_header, |
247 | .validate = ax25_validate_header, | ||
233 | }; | 248 | }; |
234 | 249 | ||
235 | EXPORT_SYMBOL(ax25_header_ops); | 250 | EXPORT_SYMBOL(ax25_header_ops); |