[v2,03/15] dpp: check configurator role in config request frame

Commit Message

James Prestwood Oct. 26, 2023, 8:26 p.m. UTC

We shouldn't ever get this frame as an enrollee, so disregard
---
 src/dpp.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Denis Kenzior Oct. 29, 2023, 10:07 p.m. UTC | #1

Hi James,

On 10/26/23 15:26, James Prestwood wrote:
> We shouldn't ever get this frame as an enrollee, so disregard
> ---
>   src/dpp.c | 3 +++
>   1 file changed, 3 insertions(+)
> 

Patch 3 & 4 applied, thanks.

Regards,
-Denis

Patch

diff --git a/src/dpp.c b/src/dpp.c
index 23b17d01..edf5b797 100644
--- a/src/dpp.c
+++ b/src/dpp.c
@@ -914,6 +914,9 @@  static void dpp_handle_config_request_frame(const struct mmpdu_header *frame,
 		return;
 	}
 
+	if (dpp->role != DPP_CAPABILITY_CONFIGURATOR)
+		return;
+
 	if (memcmp(dpp->peer_addr, frame->address_2, 6)) {
 		l_debug("Configuration request not from authenticated peer");
 		return;