Pligg Security Vulnerability - Password Change Request

Register an Account
Reply
 
Thread Tools Display Modes
  #11 (permalink)  
Old 05-27-2007, 01:54 PM
wwwSENSERELYcom's Avatar
Casual Pligger
 
Join Date: May 2007
Location: Tianjin, China
Posts: 67
yes but the god use leaves a trace in the pageviews table, where is/are the queries for that where i can add WHERE userid > 1???
Reply With Quote
  #12 (permalink)  
Old 05-27-2007, 09:35 PM
dollars5's Avatar
Pligg Donor
 
Join Date: Dec 2006
Location: India
Posts: 1,960
But where does other users see the detials of page_view table?
Even should you use the page_statistics module - it just only gets the count and not all those who saw the pages. So you can just ignore that page_view most users will not make most use of this table.
Reply With Quote
  #13 (permalink)  
Old 05-28-2007, 12:26 AM
wwwSENSERELYcom's Avatar
Casual Pligger
 
Join Date: May 2007
Location: Tianjin, China
Posts: 67
no, the page view is used in theprofile pages to show which users have viewed the profile. it's called Last 5 people to view this profile:

of course I could just delete this information showing, if I could find it, or I could just modify the query to ignore user 1
Reply With Quote
  #14 (permalink)  
Old 05-28-2007, 03:53 AM
dollars5's Avatar
Pligg Donor
 
Join Date: Dec 2006
Location: India
Posts: 1,960
Just edit last_viewers() function in /libs/pageview.php that the query there ignores admin user. Just change pv_user_id > 0 to pv_user_id > 1 - but if the site has more than 1 god user - you will have to list them all in the query.

It is a good practice to just have only 1 God user and there are other level of users like admins - which can be used to administer links etc. That should be of some help.
Reply With Quote
  #15 (permalink)  
Old 05-28-2007, 04:00 PM
New Pligger
 
Join Date: May 2007
Posts: 6
Quote:
Originally Posted by dollars5 View Post
also we added a small additional query to remove the user for top users list (where user_id > 1) - this way there is less possibility that a hacker can predict and attach on any system.
Could you give more details about this? Such as which page should I change and how to change. Thanks very much.
Reply With Quote
  #16 (permalink)  
Old 05-28-2007, 09:59 PM
dollars5's Avatar
Pligg Donor
 
Join Date: Dec 2006
Location: India
Posts: 1,960
Generally the users are listed in following places:
1] top-users: topusers.php
Code:
case 0: // sort users alphabetically
		$select = "SELECT user_id";
 		$from_where = " FROM " . table_users . " WHERE user_id != 1 ";
 		$order_by = " ORDER BY user_karma DESC ";
		break;
- so admin is not listed in the top-user

2] last_viewers() function in /libs/pageview.php - pv_user_id > 0 to pv_user_id > 1
This makes admin not shown in the who visited this profile page

If you see the users listed in other places let me know I will try to provide you a way to hide the admin.
I have not edited the who votes as the admin is reserved from voting in my proposed case
Reply With Quote
  #17 (permalink)  
Old 05-28-2007, 10:03 PM
AshDigg's Avatar
Mayor of PliggVille/Coder
 
Join Date: Dec 2005
Posts: 1,515
The 'god' users are now hidden starting with the 9.6 release.
Reply With Quote
  #18 (permalink)  
Old 05-28-2007, 10:05 PM
dollars5's Avatar
Pligg Donor
 
Join Date: Dec 2006
Location: India
Posts: 1,960
Wow - that is great news - good work Ash.

Also as pointed here Pligg Security Vulnerability - Password Change Request - God should be hidden from profile views too.

pls change the user_id to user_level in my above post.
Reply With Quote
  #19 (permalink)  
Old 05-29-2007, 05:22 PM
New Pligger
 
Join Date: May 2007
Posts: 6
Thanks dollars5. These information are really helpful.
Reply With Quote
  #20 (permalink)  
Old 06-01-2007, 02:53 PM
Pligg Donor
 
Join Date: Jan 2007
Posts: 86
In this patch i cann't modify password
Reply With Quote
Reply

Thread Tools
Display Modes


Similar Threads
Thread Thread Starter Forum Replies Last Post
Pligg Database User Name and Password... ReddyEye Questions and Comments 1 08-02-2009 11:24 AM
Security Question - Pligg Sumbitter / Auto Pligg MissDanni Questions and Comments 3 07-04-2009 11:34 AM
Pligg Donation Drive 2009 chuckroast Questions and Comments 0 03-23-2009 05:42 PM
Pligg 9.9 - Cannot Change Password - "There was a token error." sethc Questions and Comments 6 12-25-2008 11:14 AM
God cannot change his own password. vanlawrence Questions and Comments 3 04-08-2008 05:35 PM


Pligg Modules and Pligg Templates from Pligg Pro Find support on the Pligg CMS Forum - 24 hours a day! Make a donation to support Pligg CMS development