640-bridge-only-accept-EAP-locally.patch 2.2 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182
  1. From: Felix Fietkau <nbd@nbd.name>
  2. Date: Fri, 7 Jul 2017 17:18:54 +0200
  3. Subject: bridge: only accept EAP locally
  4. When bridging, do not forward EAP frames to other ports, only deliver
  5. them locally, regardless of the state.
  6. Signed-off-by: Felix Fietkau <nbd@nbd.name>
  7. [add disable_eap_hack sysfs attribute]
  8. Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com>
  9. ---
  10. --- a/net/bridge/br_input.c
  11. +++ b/net/bridge/br_input.c
  12. @@ -173,10 +173,14 @@ int br_handle_frame_finish(struct net *n
  13. }
  14. }
  15. + BR_INPUT_SKB_CB(skb)->brdev = br->dev;
  16. +
  17. + if (skb->protocol == htons(ETH_P_PAE) && !br->disable_eap_hack)
  18. + return br_pass_frame_up(skb);
  19. +
  20. if (p->state == BR_STATE_LEARNING)
  21. goto drop;
  22. - BR_INPUT_SKB_CB(skb)->brdev = br->dev;
  23. BR_INPUT_SKB_CB(skb)->src_port_isolated = !!(p->flags & BR_ISOLATED);
  24. if (IS_ENABLED(CONFIG_INET) && skb->protocol == htons(ETH_P_ARP))
  25. --- a/net/bridge/br_private.h
  26. +++ b/net/bridge/br_private.h
  27. @@ -320,6 +320,8 @@ struct net_bridge {
  28. u16 group_fwd_mask;
  29. u16 group_fwd_mask_required;
  30. + bool disable_eap_hack;
  31. +
  32. /* STP */
  33. bridge_id designated_root;
  34. bridge_id bridge_id;
  35. --- a/net/bridge/br_sysfs_br.c
  36. +++ b/net/bridge/br_sysfs_br.c
  37. @@ -170,6 +170,30 @@ static ssize_t group_fwd_mask_store(stru
  38. }
  39. static DEVICE_ATTR_RW(group_fwd_mask);
  40. +static ssize_t disable_eap_hack_show(struct device *d,
  41. + struct device_attribute *attr,
  42. + char *buf)
  43. +{
  44. + struct net_bridge *br = to_bridge(d);
  45. + return sprintf(buf, "%u\n", br->disable_eap_hack);
  46. +}
  47. +
  48. +static int set_disable_eap_hack(struct net_bridge *br, unsigned long val)
  49. +{
  50. + br->disable_eap_hack = !!val;
  51. +
  52. + return 0;
  53. +}
  54. +
  55. +static ssize_t disable_eap_hack_store(struct device *d,
  56. + struct device_attribute *attr,
  57. + const char *buf,
  58. + size_t len)
  59. +{
  60. + return store_bridge_parm(d, buf, len, set_disable_eap_hack);
  61. +}
  62. +static DEVICE_ATTR_RW(disable_eap_hack);
  63. +
  64. static ssize_t priority_show(struct device *d, struct device_attribute *attr,
  65. char *buf)
  66. {
  67. @@ -817,6 +841,7 @@ static struct attribute *bridge_attrs[]
  68. &dev_attr_ageing_time.attr,
  69. &dev_attr_stp_state.attr,
  70. &dev_attr_group_fwd_mask.attr,
  71. + &dev_attr_disable_eap_hack.attr,
  72. &dev_attr_priority.attr,
  73. &dev_attr_bridge_id.attr,
  74. &dev_attr_root_id.attr,