Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open Redirect In Login #1333

Open
kajalNair opened this issue Oct 9, 2020 · 9 comments
Open

Open Redirect In Login #1333

kajalNair opened this issue Oct 9, 2020 · 9 comments

Comments

@kajalNair
Copy link

I followed your previous issues and found that it has been already disclosed and closed as "mitigated" but I am still able to reproduce it.
In this scenario, an attacker can directly append "redirect" parameter to the login page URL request and after successful login, it redirects the user to arbitrary page.
The vulnerability is present because application still looks for redirect parameter in URL (in GetSimpleCMS-master\admin\inc\template_functions.php file) and if present, sets the redirection to specified path.

redirect

@kajalNair
Copy link
Author

The vulnerable function:

function gotoDefaultPage(){
	if (isset($_GET['redirect'])) redirect(htmlentities($_GET['redirect']));
	else redirect(getDef('GSDEFAULTPAGE'));

}

@tablatronix
Copy link
Member

#1300

What version is this against?

@kajalNair
Copy link
Author

2020-10-09 22_02_39-User Profile · testcms com

@kajalNair
Copy link
Author

3.4.0 version

@tablatronix
Copy link
Member

ok, thanks

Looks like this was only hotfixed, 3.x

@tablatronix tablatronix added this to the 3.4.0 milestone Oct 9, 2020
@kajalNair
Copy link
Author

Does this qualify for a CVE?

@tablatronix
Copy link
Member

3.4 is alpha, so not release. Probably not

@tablatronix
Copy link
Member

@kajalNair
Copy link
Author

okay, cool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants