[PATCH 05/24] auth: Remove obsolete trace message from parse_form()

Kevin Cernekee cernekee at gmail.com
Sat Nov 3 13:22:47 EDT 2012


parse_form() used to add hidden form fields to the query string on
the spot, but since it doesn't anymore, it should not display the
"Fixed options" trace message.

After making this change, we can remove the request / request_body
arguments.

Signed-off-by: Kevin Cernekee <cernekee at gmail.com>
---
 auth.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/auth.c b/auth.c
index 4c9d623..45507ce 100644
--- a/auth.c
+++ b/auth.c
@@ -172,7 +172,7 @@ static int parse_auth_choice(struct openconnect_info *vpninfo, struct oc_auth_fo
  *  = 1, when form was parsed
  */
 static int parse_form(struct openconnect_info *vpninfo, struct oc_auth_form *form,
-		      xmlNode *xml_node, char *body, int bodylen)
+		      xmlNode *xml_node)
 {
 	char *input_type, *input_name, *input_label;
 
@@ -255,8 +255,6 @@ static int parse_form(struct openconnect_info *vpninfo, struct oc_auth_form *for
 		*p = opt;
 	}
 
-	vpn_progress(vpninfo, PRG_TRACE, _("Fixed options give %s\n"), body);
-
 	return 0;
 }
 
@@ -395,7 +393,7 @@ int parse_xml_response(struct openconnect_info *vpninfo, char *response,
 			}
 			vpninfo->redirect_url = strdup(form->action);
 
-			ret = parse_form(vpninfo, form, xml_node, request_body, req_len);
+			ret = parse_form(vpninfo, form, xml_node);
 			if (ret < 0)
 				goto out;
 		} else if (!vpninfo->csd_scriptname && !strcmp((char *)xml_node->name, "csd")) {
-- 
1.7.10.4




More information about the openconnect-devel mailing list