auth: Remove obsolete trace message from parse_form()
authorKevin Cernekee <cernekee@gmail.com>
Fri, 26 Oct 2012 04:16:57 +0000 (21:16 -0700)
committerKevin Cernekee <cernekee@gmail.com>
Sun, 28 Oct 2012 05:10:03 +0000 (22:10 -0700)
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@gmail.com>
auth.c

diff --git a/auth.c b/auth.c
index 4c9d623..45507ce 100644 (file)
--- 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")) {