diff --git a/admin/roles/allowassign.php b/admin/roles/allowassign.php index 70bb0f91447..f1d6a8a6402 100755 --- a/admin/roles/allowassign.php +++ b/admin/roles/allowassign.php @@ -21,10 +21,10 @@ $roles = get_all_roles(); - if ($grant = data_submitted()) { + if ($grant = data_submitted() and confirm_sesskey()) { foreach ($grant as $grole => $val) { - if ($grole == 'dummy') { + if ($grole == 'dummy' or !strpos($grole, '_')) { continue; } @@ -81,6 +81,7 @@ print_table($table); echo '
'; echo ''; // this is needed otherwise we do not know a form has been submitted + echo ''; echo '
'; admin_externalpage_print_footer(); diff --git a/admin/roles/allowoverride.php b/admin/roles/allowoverride.php index 65fa38235b4..0f5da50500f 100755 --- a/admin/roles/allowoverride.php +++ b/admin/roles/allowoverride.php @@ -19,10 +19,10 @@ $roles = get_all_roles(); - if ($grant = data_submitted()) { + if ($grant = data_submitted() and confirm_sesskey()) { foreach ($grant as $grole => $val) { - if ($grole == 'dummy') { + if ($grole == 'dummy' or !strpos($grole, '_')) { continue; } @@ -79,6 +79,7 @@ print_table($table); echo '
'; echo ''; // this is needed otherwise we do not know a form has been submitted + echo ''; echo '
'; admin_externalpage_print_footer();