[PATCH] dpp: fix removed dpp_reset in Stop()

[PATCH] dpp: fix removed dpp_reset in Stop()

[James Prestwood]

It seems in my patch reordering both stop methods lost the actual
call to dpp_reset().
---
 src/dpp.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/dpp.c b/src/dpp.c
index a0c44eda..9501f3de 100644
--- a/src/dpp.c
+++ b/src/dpp.c
@@ -3961,9 +3961,13 @@ static struct l_dbus_message *dpp_dbus_stop(struct l_dbus *dbus,
 {
 	struct dpp_sm *dpp = user_data;
 
+	l_debug("");
+
 	if (dpp->interface != DPP_INTERFACE_DPP)
 		return dbus_error_not_found(message);
 
+	dpp_reset(dpp);
+
 	return l_dbus_message_new_method_return(message);
 }
 
@@ -4340,9 +4344,13 @@ static struct l_dbus_message *dpp_dbus_pkex_stop(struct l_dbus *dbus,
 {
 	struct dpp_sm *dpp = user_data;
 
+	l_debug("");
+
 	if (dpp->interface != DPP_INTERFACE_PKEX)
 		return dbus_error_not_found(message);
 
+	dpp_reset(dpp);
+
 	return l_dbus_message_new_method_return(message);
 }
 
-- 
2.25.1

Re: [PATCH] dpp: fix removed dpp_reset in Stop()

[Denis Kenzior]

Hi James,

On 11/9/23 11:25, James Prestwood wrote:
> It seems in my patch reordering both stop methods lost the actual
> call to dpp_reset().

In the future, lets use the Fixes: tag for commits like this.

> ---
>   src/dpp.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 

Applied, thanks.

Regards,
-Denis