Store copy of hash instead of working on live object
This avoids possible race conditions when doing the checks. Signed-off-by: Michal Čihař <michal@cihar.com>
This commit is contained in:
parent
f2add98bd9
commit
c2f7a898dd
11
js/ajax.js
11
js/ajax.js
@ -711,15 +711,16 @@ AJAX.setUrlHash = (function (jQuery, window) {
|
||||
/**
|
||||
* Start initialisation
|
||||
*/
|
||||
if (window.location.hash.substring(0, 8) == '#PMAURL-') {
|
||||
var urlhash = window.location.hash;
|
||||
if (urlhash.substring(0, 8) == '#PMAURL-') {
|
||||
// We have a valid hash, let's redirect the user
|
||||
// to the page that it's pointing to
|
||||
var colon_position = window.location.hash.indexOf(':');
|
||||
var questionmark_position = window.location.hash.indexOf('?');
|
||||
var colon_position = urlhash.indexOf(':');
|
||||
var questionmark_position = urlhash.indexOf('?');
|
||||
if (colon_position != -1 && questionmark_position != -1 && colon_position < questionmark_position) {
|
||||
var hash_url = window.location.hash.substring(colon_position + 1, questionmark_position);
|
||||
var hash_url = urlhash.substring(colon_position + 1, questionmark_position);
|
||||
if (PMA_gotoWhitelist.indexOf(hash_url) != -1) {
|
||||
window.location = window.location.hash.substring(
|
||||
window.location = urlhash.substring(
|
||||
colon_position + 1
|
||||
);
|
||||
}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user