Closed (fixed)
Project:
Payment
Version:
7.x-1.2
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Dec 2012 at 18:39 UTC
Updated:
24 Jul 2013 at 08:46 UTC
I just noticed some user can see other payments (even if they doesn't have that permission). I found this problem in payment_access function:
return user_access('payment.payment.' . $operation . '.any', $account) || $payment && user_access('payment.payment.' . $operation . '.own', $account) && $account->uid = $payment->uid;
$account->uid == $payment->uid should be the correct sintaxis.
This cause some session exchange between my users, so I think this is critical.
Comments
Comment #1
xanoThis issue was fixed in collaboration with the security team. See SA-CONTRIB-2013-002 for more details.
Comment #3
xano