Closed (fixed)
Project:
UC Recurring Payments and Subscriptions
Version:
7.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Sep 2012 at 11:37 UTC
Updated:
26 Mar 2014 at 23:31 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
VarnavskiyAI commentedSimple, but not optimal fix.
Comment #2
boon4376 commentedI would like to report that this huge security issue still exists for
Ubercart 6.x-2.10
UC Recurring Payments and Subscriptions 6.x-2.0-beta2
The workaround is to disable view own recurring fees under permissions - however, users will be unable to update their own payment information.
Since this issue has persisted from Beta1 - Beta2, I am curious if the developer is working on a fix for this?
Comment #3
rzamorski commentedCan anyone verify that the fix posted in #1 did, in fact, work?
Comment #4
boon4376 commentedMy basic testing of playing with URL variables after applying fix #1 shows that it does work to prevent users from viewing other recurring fee pages.
I have also tested manipulating the form action code to another /user/37/recurring/XX variable to ensure that users cannot still hack this to update other's payment information.
action="/user/37/recurring/2/update/authorizenet_cim?destination=user/37/recurring-fees"So the fix appears to be working fine for me for both preventing users from viewing and changing payment information for other users.
Comment #5
M. George Hansen commentedThis is also an issue in 7.x-2.x-dev. I've attached a patch that implements a slightly refactored version of VarnavskiyAI's changes in #1. I've done some light testing and this patch seems to solve the issue.
Comment #6
univate commentedCommitted to both 6.x and 7.x