diff options
| author | Kai Blin <kai@samba.org> | 2011-07-08 15:04:48 +0200 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2011-07-24 21:27:23 +0200 |
| commit | 9839935c29ec0ab522994436e6e89939696409de (patch) | |
| tree | 486659d298ca96ea83957e08761bbaab7f62857f | |
| parent | 6ea5fac27f2fef35ea12c24250948e00245aacee (diff) | |
| download | samba-9839935c29ec0ab522994436e6e89939696409de.tar.gz | |
s3 swat: Add XSRF protection to shares page
Signed-off-by: Kai Blin <kai@samba.org>
| -rw-r--r-- | source3/web/swat.c | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/source3/web/swat.c b/source3/web/swat.c index 5dab7c78815..f1f226ba9c7 100644 --- a/source3/web/swat.c +++ b/source3/web/swat.c @@ -983,11 +983,17 @@ static void shares_page(void) int mode = 0; unsigned int parm_filter = FLAG_BASIC; size_t converted_size; + const char form_name[] = "shares"; + + printf("<H2>%s</H2>\n", _("Share Parameters")); + + if (!verify_xsrf_token(form_name)) { + goto output_page; + } if (share) snum = lp_servicenumber(share); - printf("<H2>%s</H2>\n", _("Share Parameters")); if (cgi_variable("Commit") && snum >= 0) { commit_parameters(snum); @@ -1013,10 +1019,6 @@ static void shares_page(void) } } - printf("<FORM name=\"swatform\" method=post>\n"); - - printf("<table>\n"); - if ( cgi_variable("ViewMode") ) mode = atoi(cgi_variable_nonull("ViewMode")); if ( cgi_variable("BasicMode")) @@ -1024,6 +1026,12 @@ static void shares_page(void) if ( cgi_variable("AdvMode")) mode = 1; +output_page: + printf("<FORM name=\"swatform\" method=post>\n"); + print_xsrf_token(cgi_user_name(), cgi_user_pass(), form_name); + + printf("<table>\n"); + ViewModeBoxes( mode ); switch ( mode ) { case 0: |