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

DB_Parameter_Tampering @ /BookDetail.cs #26

Open
apcxtest opened this issue Nov 20, 2023 · 0 comments
Open

DB_Parameter_Tampering @ /BookDetail.cs #26

apcxtest opened this issue Nov 20, 2023 · 0 comments

Comments

@apcxtest
Copy link
Owner

apcxtest commented Nov 20, 2023

Checkmarx (SAST): DB_Parameter_Tampering
Security Issue: Read More about DB_Parameter_Tampering
Applications: test_App
Checkmarx Project: apcxtest/test-repo-pub2
Repository URL: https://github.com/apcxtest/test-repo-pub2
Branch: main
Scan ID: 5072abd4-8e3e-49e3-b1b5-44426e72387c


Method Order_Show at line 314 of /BookDetail.cs gets user input from element Value. This input is used by the application, without being validated, to filter personal records from sensitive database tables. Method Order_Show submits a query to the database Fill, at line 323 of /BookDetail.cs, without any additional filtering by the database. This could allow the user to choose different records based on the id.

Result #1:
Severity: MEDIUM
State: TO_VERIFY
Status: RECURRENT
Attack Vector:

    1. Value: /BookDetail.cs[314,60]
    2. ToSQL: /BookDetail.cs[314,37]
    3. sWhere: /BookDetail.cs[314,3]
    4. sWhere: /BookDetail.cs[318,49]
    5. sSQL: /BookDetail.cs[318,10]
    6. sSQL: /BookDetail.cs[319,53]
    7. OleDbDataAdapter: /BookDetail.cs[319,32]
    8. dsCommand: /BookDetail.cs[319,20]
    9. dsCommand: /BookDetail.cs[323,7]
    10. Fill: /BookDetail.cs[323,17]
    Review result in Checkmarx One: DB_Parameter_Tampering

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

1 participant