Thanks for reminding me that there was something pending here as well...
This is a bit complicated, though. The bugfix needs to go to 2.6 as
well, so "2/2 applied", but the v2 as sent to the list has a hunk that
is really a bugfix for get_key_by_management_key_id(), which is in
1/2, and that is refactoring -> master.
So I've only applied the first hunk here, and the second hunk should
go together with all of 1/2 into a "master only" patch. Right?
I have run this through the basic client test, which isn't exactly
excercising the management interface (not at all) but I have nothing
for that - so I'm relying on Selva's ACK and Jemmy Wang's test
results in GH #359 on "does it actually fix the bug".
Your patch has been applied to the master and release/2.6 branch.
commit 223baa9c9b818e4c542a9037f190f53ce6f7af5c (master)
commit 0caf0389454c75ae9dfeda096a820446d2d1a926 (HEAD -> release/2.6)
Author: Arne Schwabe
Date: Mon May 22 12:11:38 2023 +0200
Fix CR_RESPONSE mangaement message using wrong key_id
Signed-off-by: Arne Schwabe <a...@rfc2549.org>
Acked-by: Selva Nair <selva.n...@gmail.com>
Message-Id: <20230522101138.2842378-2-a...@rfc2549.org>
URL:
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26719.html
Signed-off-by: Gert Doering <g...@greenie.muc.de>
--
kind regards,
Gert Doering
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
