X-Git-Url: http://www.privoxy.org/gitweb/?p=privoxy.git;a=blobdiff_plain;f=cgisimple.c;h=5ced54c938d568acc7b9d13af29fac75231f3949;hp=35284a08ecf1290bfed44ba7a0ea29dbf2d1cace;hb=8d60ab1f19850c964372b3c01c3d1fa82ac6f579;hpb=377bb95c67b8ad7a705691695297fd08e43901b6 diff --git a/cgisimple.c b/cgisimple.c index 35284a08..5ced54c9 100644 --- a/cgisimple.c +++ b/cgisimple.c @@ -1,4 +1,4 @@ -const char cgisimple_rcs[] = "$Id: cgisimple.c,v 1.59 2007/10/19 16:42:36 fabiankeil Exp $"; +const char cgisimple_rcs[] = "$Id: cgisimple.c,v 1.65 2008/02/23 16:33:43 fabiankeil Exp $"; /********************************************************************* * * File : $Source: /cvsroot/ijbswa/current/cgisimple.c,v $ @@ -36,6 +36,27 @@ const char cgisimple_rcs[] = "$Id: cgisimple.c,v 1.59 2007/10/19 16:42:36 fabian * * Revisions : * $Log: cgisimple.c,v $ + * Revision 1.65 2008/02/23 16:33:43 fabiankeil + * Let forward_url() use the standard parameter ordering + * and mark its second parameter immutable. + * + * Revision 1.64 2008/02/03 13:56:07 fabiankeil + * Add SOCKS5 support for show-url-info CGI page. + * + * Revision 1.63 2008/02/01 06:04:31 fabiankeil + * If edit buttons on the show-url-info CGI page are hidden, explain why. + * + * Revision 1.62 2008/02/01 05:52:40 fabiankeil + * Hide edit buttons on the show-url-info CGI page if enable-edit-action + * is disabled. Patch by Lee with additional white space adjustments. + * + * Revision 1.61 2008/01/26 11:13:25 fabiankeil + * If enable-edit-actions is disabled, hide the edit buttons and explain why. + * + * Revision 1.60 2007/10/27 13:12:13 fabiankeil + * Finish 1.49 and check write access before + * showing edit buttons on show-url-info page. + * * Revision 1.59 2007/10/19 16:42:36 fabiankeil * Plug memory leak I introduced five months ago. * Yay Valgrind and Privoxy-Regression-Test. @@ -1240,7 +1261,9 @@ jb_err cgi_show_status(struct client_state *csp, if (!err) err = string_append(&s, buf); #ifdef FEATURE_CGI_EDIT_ACTIONS - if (NULL == strstr(csp->actions_list[i]->filename, "standard.action") && NULL != csp->config->actions_file_short[i]) + if ((csp->config->feature_flags & RUNTIME_FEATURE_CGI_EDIT_ACTIONS) + && (NULL == strstr(csp->actions_list[i]->filename, "standard.action")) + && (NULL != csp->config->actions_file_short[i])) { #ifdef HAVE_ACCESS if (access(csp->config->actions_file[i], W_OK) == 0) @@ -1311,6 +1334,13 @@ jb_err cgi_show_status(struct client_state *csp, if (!err) err = map_block_killer(exports, "trust-support"); #endif /* ndef FEATURE_TRUST */ +#ifdef FEATURE_CGI_EDIT_ACTIONS + if (!err && (csp->config->feature_flags & RUNTIME_FEATURE_CGI_EDIT_ACTIONS)) + { + err = map_block_killer(exports, "cgi-editor-is-disabled"); + } +#endif /* ndef CGI_EDIT_ACTIONS */ + if (err) { free_map(exports); @@ -1543,20 +1573,24 @@ jb_err cgi_show_url_info(struct client_state *csp, string_append(&matches, buf); string_append(&matches, "View"); #ifdef FEATURE_CGI_EDIT_ACTIONS -#ifdef HAVE_ACCESS - if (access(csp->config->actions_file[i], W_OK) == 0) + if (csp->config->feature_flags & RUNTIME_FEATURE_CGI_EDIT_ACTIONS) { +#ifdef HAVE_ACCESS + if (access(csp->config->actions_file[i], W_OK) == 0) + { #endif /* def HAVE_ACCESS */ - snprintf(buf, sizeof(buf), " ", i); - string_append(&matches, buf); - string_append(&matches, "Edit"); + snprintf(buf, sizeof(buf), + " ", i); + string_append(&matches, buf); + string_append(&matches, "Edit"); #ifdef HAVE_ACCESS - } - else - { - string_append(&matches, " No write access."); - } + } + else + { + string_append(&matches, " No write access."); + } #endif /* def HAVE_ACCESS */ + } #endif /* FEATURE_CGI_EDIT_ACTIONS */ string_append(&matches, "\n"); @@ -1606,7 +1640,7 @@ jb_err cgi_show_url_info(struct client_state *csp, * but luckily it's no longer required later on anyway. */ free_current_action(csp->action); - url_actions(url_to_query, csp); + get_url_actions(csp, url_to_query); /* * Fill in forwarding settings. @@ -1622,7 +1656,7 @@ jb_err cgi_show_url_info(struct client_state *csp, * display the proxy port and an eventual second forwarder. */ { - const struct forward_spec * fwd = forward_url(url_to_query, csp); + const struct forward_spec *fwd = forward_url(csp, url_to_query); if ((fwd->gateway_host == NULL) && (fwd->forward_host == NULL)) { @@ -1637,8 +1671,24 @@ jb_err cgi_show_url_info(struct client_state *csp, if (fwd->gateway_host != NULL) { - if (!err) err = map(exports, "socks-type", 1, (fwd->type == SOCKS_4) ? - "socks4" : "socks4a", 1); + char *socks_type = NULL; + + switch (fwd->type) + { + case SOCKS_4: + socks_type = "socks4"; + break; + case SOCKS_4A: + socks_type = "socks4a"; + break; + case SOCKS_5: + socks_type = "socks5"; + break; + default: + log_error(LOG_LEVEL_FATAL, "Unknown socks type: %d.", fwd->type); + } + + if (!err) err = map(exports, "socks-type", 1, socks_type, 1); if (!err) err = map(exports, "gateway-host", 1, fwd->gateway_host, 1); snprintf(port, sizeof(port), "%d", fwd->gateway_port); if (!err) err = map(exports, "gateway-port", 1, port, 1); @@ -1670,7 +1720,14 @@ jb_err cgi_show_url_info(struct client_state *csp, return JB_ERR_MEMORY; } - if (map(exports, "matches", 1, matches , 0)) +#ifdef FEATURE_CGI_EDIT_ACTIONS + if ((csp->config->feature_flags & RUNTIME_FEATURE_CGI_EDIT_ACTIONS)) + { + err = map_block_killer(exports, "cgi-editor-is-disabled"); + } +#endif /* FEATURE_CGI_EDIT_ACTIONS */ + + if (err || map(exports, "matches", 1, matches , 0)) { free_current_action(action); free_map(exports);