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

fix: login into another accounts #2853

Merged
merged 9 commits into from
Sep 14, 2024
Merged

fix: login into another accounts #2853

merged 9 commits into from
Sep 14, 2024

Conversation

phacUFPE
Copy link
Contributor

@phacUFPE phacUFPE commented Aug 26, 2024

Description

Prevents the login to other account when injecting a new login.php.

Behaviour

Actual

Change you login.php to request the players from a static account id that is not your account and try login into the server, it should login in a character that does not belong to your account.

Expected

Change you login.php to request the players from a static account id that is not your account and try login into the server, it shouldn't login in a character that does not belong to your account.

Fixes #2847

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Change your login.php to list a static account id different of which you are logging in.
  • Try login into a character using your account and it should not be able to.

Test Configuration:

  • Server Version: 3.1.2 (latest)
  • Client: 13.40
  • Operating System: Windows

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I checked the PR checks reports
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Copy link
Contributor

Qodana for C/C++

2 new problems were found

Inspection name Severity Problems
misra-cpp2008-5-3-1 🔶 Warning 1
function-cognitive-complexity 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/[email protected]
        with:
          upload-result: true
Contact Qodana team

Contact us at [email protected]

@phacUFPE phacUFPE merged commit b3b19a6 into main Sep 14, 2024
37 of 38 checks passed
@phacUFPE phacUFPE deleted the fix_login_another_account branch September 14, 2024 20:58
PrimosOT pushed a commit to PrimosOT/canary that referenced this pull request Sep 18, 2024
Fixes the login to other account when injecting a custom login.php
s2leandro155 pushed a commit to s2leandro155/rzyns that referenced this pull request Sep 26, 2024
Fixes the login to other account when injecting a custom login.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Players gaining access to other characters
6 participants