From da70b2d1c21de9dc495df42e4903ee9950d6bed8 Mon Sep 17 00:00:00 2001
From: Marcel Holtmann <marcel@holtmann.org>
Date: Sun, 14 Oct 2018 07:55:29 +0200
Subject: [PATCH] wired: The supplicant always uses PAE group address for
 transmission

---
 wired/ethdev.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/wired/ethdev.c b/wired/ethdev.c
index 331478b8..c7db8e6c 100644
--- a/wired/ethdev.c
+++ b/wired/ethdev.c
@@ -65,7 +65,6 @@ struct ethdev {
 struct eapol {
 	struct ethdev *dev;
 	uint8_t addr[ETH_ALEN];
-	bool use_group_addr;
 	struct eap_state *eap;
 	struct l_settings *cred;
 };
@@ -154,10 +153,11 @@ static void eap_tx_packet(const uint8_t *eap_data, size_t len, void *user_data)
 	l_put_be16(len, frame + 2);
 	memcpy(frame + 4, eap_data, len);
 
-	if (eapol->use_group_addr)
-		pae_write(eapol->dev, pae_group_addr, frame, len + 4);
-	else
-		pae_write(eapol->dev, eapol->addr, frame, len + 4);
+	/*
+	 * The supplicant / client always uses the PAE group address for
+	 * sending the EAP packets.
+	 */
+	pae_write(eapol->dev, pae_group_addr, frame, len + 4);
 }
 
 static void eap_complete(enum eap_result result, void *user_data)
@@ -203,7 +203,6 @@ static void rx_packet(struct ethdev *dev, const uint8_t *addr,
 			eapol = l_new(struct eapol, 1);
 			eapol->dev = dev;
 			memcpy(eapol->addr, addr, ETH_ALEN);
-			eapol->use_group_addr = true;
 			eapol->eap = eap_new(eap_tx_packet,
 							eap_complete, eapol);
 			if (!eapol->eap) {