Improper authorization - receptionist can read all secure messaging in openemr/openemr

Valid

Reported on

Apr 23rd 2022


Description

Hi there openemr maintainers, I would like to report an improper authorization vulnerability in your source code.

Proof of Concept

  1. Install openemr in your system and create an admin account and a receptionist account
  2. Use admin account and create a secure message by go to Portal > Portal Mail
  3. Log in as receptionist and go to Portal > Portal Mail, you can see that you see no secure message; the request body for portal mail looks like
task=getdeleted&owner=receptionist

Replace this with owner=admin and you can view all messages by admin user.

Impact

Allow receptionist to read all messages.

We are processing your report and will contact the openemr team within 24 hours. 4 months ago
We have contacted a member of the openemr team and are waiting to hear back 4 months ago
We have sent a follow up to the openemr team. We will try again in 7 days. 4 months ago
We have sent a second follow up to the openemr team. We will try again in 10 days. 3 months ago
openemr/openemr maintainer validated this vulnerability 3 months ago
justinp09010 has been awarded the disclosure bounty
The fix bounty is now up for grabs
The researcher's credibility has increased: +7
openemr/openemr maintainer
3 months ago

Maintainer


A preliminary fix for this has been placed in our development codebase at following 2 commits: https://github.com/openemr/openemr/commit/81ffc7288b0b9a38e05bd78c9d6a52dd643ee92d https://github.com/openemr/openemr/commit/96b1684a02d43a8c943cfe3e8c7caba51d36dbfb

The fix will officially be released in the next OpenEMR 6.1.0 patch 2 (6.1.0.2). After we release this patch, I will then mark this item as fixed (probably in about a month).

We have sent a fix follow up to the openemr team. We will try again in 7 days. 3 months ago
We have sent a second fix follow up to the openemr team. We will try again in 10 days. 3 months ago
We have sent a third and final fix follow up to the openemr team. This report is now considered stale. 2 months ago
openemr/openemr maintainer confirmed that a fix has been merged on 81ffc7 21 days ago
The fix bounty has been dropped
openemr/openemr maintainer
21 days ago

Maintainer


This fix was included in OpenEMR version 7.0.0, which was recently released.

to join this conversation