[Openvpn-devel,v1] manage: use man argument in management_query_cert()

Message ID 20261102170041.110861-1-frank@lichtenheld.com
State New
Headers
Series [Openvpn-devel,v1] manage: use man argument in management_query_cert() |

Commit Message

Frank Lichtenheld Nov. 2, 2026, 5 p.m. UTC
  From: Razvan Cojocaru <razvanc@mailbox.org>

It passed the global management to
management_query_multiline_flatten_newline() while using its man
argument for the state and input buffers. No functional change, the
only caller passes the global.

Change-Id: Ifc12dafe9621ca00dc0022ad426afebcf38fbc3b
Signed-off-by: Razvan Cojocaru <razvanc@mailbox.org>
Acked-by: Frank Lichtenheld <frank@lichtenheld.com>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2094
---

This change was reviewed on Gerrit and approved by at least one
developer. I request to merge it to master.

Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2094
This mail reflects revision 1 of this Change.

Acked-by according to Gerrit (reflected above):
Frank Lichtenheld <frank@lichtenheld.com>
  

Patch

diff --git a/src/openvpn/manage.c b/src/openvpn/manage.c
index 16864c8..d391559 100644
--- a/src/openvpn/manage.c
+++ b/src/openvpn/manage.c
@@ -3937,8 +3937,8 @@ 
 
     char *result;
     result = management_query_multiline_flatten_newline(
-        management, NULL, (char *)buf_bptr(&buf_prompt), "certificate",
-        &man->connection.ext_cert_state, &man->connection.ext_cert_input);
+        man, NULL, (char *)buf_bptr(&buf_prompt), "certificate", &man->connection.ext_cert_state,
+        &man->connection.ext_cert_input);
     free_buf(&buf_prompt);
     return result;
 }