diff --git a/webtools/bugzilla/template/en/default/account/prefs/prefs.html.tmpl b/webtools/bugzilla/template/en/default/account/prefs/prefs.html.tmpl index d455b5b66a4f..327218e170ea 100644 --- a/webtools/bugzilla/template/en/default/account/prefs/prefs.html.tmpl +++ b/webtools/bugzilla/template/en/default/account/prefs/prefs.html.tmpl @@ -100,7 +100,16 @@
- [% PROCESS "account/prefs/${current_tab.name}.html.tmpl" IF current_tab.name.defined %] + [%# See bug 160710 for why this is done this way %] + [% IF current_tab.name == "account" %] + [% PROCESS "account/prefs/account.html.tmpl" %] + [% ELSIF current_tab.name == "email" %] + [% PROCESS "account/prefs/email.html.tmpl" %] + [% ELSIF current_tab.name == "footer" %] + [% PROCESS "account/prefs/footer.html.tmpl" %] + [% ELSIF current_tab.name == "permissions" %] + [% PROCESS "account/prefs/permissions.html.tmpl" %] + [% END %] [% IF current_tab.saveable %] diff --git a/webtools/bugzilla/userprefs.cgi b/webtools/bugzilla/userprefs.cgi index 84c8ea027978..cf58cf2652ea 100755 --- a/webtools/bugzilla/userprefs.cgi +++ b/webtools/bugzilla/userprefs.cgi @@ -370,6 +370,9 @@ $vars->{'changes_saved'} = $::FORM{'dosave'}; my $current_tab_name = $::FORM{'tab'} || "account"; +# The SWITCH below makes sure that this is valid +trick_taint($current_tab_name); + $vars->{'current_tab_name'} = $current_tab_name; # Do any saving, and then display the current tab. @@ -393,6 +396,8 @@ SWITCH: for ($current_tab_name) { DoPermissions(); last SWITCH; }; + ThrowUserError("Unknown tab " . html_quote($current_tab_name) + . ""); } # Generate and return the UI (HTML page) from the appropriate template.