Skip to content
This repository has been archived by the owner on Mar 23, 2021. It is now read-only.

Stop code execution if user is not authenticated #154

Merged
merged 1 commit into from
Sep 27, 2020
Merged

Stop code execution if user is not authenticated #154

merged 1 commit into from
Sep 27, 2020

Conversation

sunny0day
Copy link
Contributor

Without an exit statement after the 302 response header is set, all code after including this file, will be executed.

This is a major security vulnerability, because one would be able to, for example:

  • Add system users
  • Change the password of existing users
  • Enable/disable SSH
  • Enable/disable the firewall

as an unauthenticated user.

@duncan-mccracken duncan-mccracken merged commit e422566 into jamf:master Sep 27, 2020
@sunny0day sunny0day deleted the patch-2 branch September 28, 2020 11:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants