diff options
| author | Kai Blin <kai@samba.org> | 2011-07-08 15:06:13 +0200 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2011-07-26 20:48:13 +0200 |
| commit | e4fe62ff8d558f3f2bfe22fd880f76e69162e2f8 (patch) | |
| tree | a81847c9f9e28ff8aaff3c189db5f625a1f2a6ff | |
| parent | 78bee109191146c10bb0fd751dfa845d4796668d (diff) | |
| download | samba-e4fe62ff8d558f3f2bfe22fd880f76e69162e2f8.tar.gz | |
s3 swat: Add XSRF protection to printer page
Signed-off-by: Kai Blin <kai@samba.org>
(cherry picked from commit deb66470413780c93656294a1dca40f8cc1bada8)
| -rw-r--r-- | source3/web/swat.c | 28 |
1 files changed, 18 insertions, 10 deletions
diff --git a/source3/web/swat.c b/source3/web/swat.c index c46cd8f592d..8fe2aec6fa6 100644 --- a/source3/web/swat.c +++ b/source3/web/swat.c @@ -1333,18 +1333,15 @@ static void printers_page(void) int i; int mode = 0; unsigned int parm_filter = FLAG_BASIC; + const char form_name[] = "printers"; + + if (!verify_xsrf_token(form_name)) { + goto output_page; + } if (share) snum = lp_servicenumber(share); - printf("<H2>%s</H2>\n", _("Printer Parameters")); - - printf("<H3>%s</H3>\n", _("Important Note:")); - printf("%s",_("Printer names marked with [*] in the Choose Printer drop-down box ")); - printf("%s",_("are autoloaded printers from ")); - printf("<A HREF=\"/swat/help/smb.conf.5.html#printcapname\" target=\"docs\">%s</A>\n", _("Printcap Name")); - printf("%s\n", _("Attempting to delete these printers from SWAT will have no effect.")); - if (cgi_variable("Commit") && snum >= 0) { commit_parameters(snum); if (snum >= iNumNonAutoPrintServices) @@ -1373,8 +1370,6 @@ static void printers_page(void) } } - printf("<FORM name=\"swatform\" method=post>\n"); - if ( cgi_variable("ViewMode") ) mode = atoi(cgi_variable_nonull("ViewMode")); if ( cgi_variable("BasicMode")) @@ -1382,6 +1377,19 @@ static void printers_page(void) if ( cgi_variable("AdvMode")) mode = 1; +output_page: + printf("<H2>%s</H2>\n", _("Printer Parameters")); + + printf("<H3>%s</H3>\n", _("Important Note:")); + printf("%s",_("Printer names marked with [*] in the Choose Printer drop-down box ")); + printf("%s",_("are autoloaded printers from ")); + printf("<A HREF=\"/swat/help/smb.conf.5.html#printcapname\" target=\"docs\">%s</A>\n", _("Printcap Name")); + printf("%s\n", _("Attempting to delete these printers from SWAT will have no effect.")); + + + printf("<FORM name=\"swatform\" method=post>\n"); + print_xsrf_token(cgi_user_name(), cgi_user_pass(), form_name); + ViewModeBoxes( mode ); switch ( mode ) { case 0: |