vBulletin Mods

The Official vBulletin Modifications Site
https://www.vbulletin.org/forum/showthread.php?t=323227

Trouble with Custom Script
by kerrghann
12 Aug 2016 09:59

So...I decided to improve on the default passwordcheck.php in the admincp. Rather then just checking to see if my users passwords are just their username, I decided to modify it to use an array that I've filled with the top 25 most commonly used passwords.

And it works! Kind of. It tells you the correct number of users (I assume) that meet the check criteria, however, for some reason when you reset those users passwords, the script continues past the number of people that it says have the vulnerable passwords and keeps climbing up and up, resetting passwords until I stop it.

Anyone feel up for helping?


Block Disabled:      (Update License Status)  
Suspended or Unlicensed Members Cannot View Code.


nhawk 12 Aug 2016 11:10

$user can't be used twice the way you're using it. The while loop will never end.


Block Disabled:      (Update License Status)  
Suspended or Unlicensed Members Cannot View Code.


kerrghann 12 Aug 2016 22:52

Oddly enough that isn't a part that I modified. That's part of the normal vbulletin script. It also works fine and doesn't go on forever, at least until I add my part:

My parts are these:


Block Disabled:      (Update License Status)  
Suspended or Unlicensed Members Cannot View Code.

and


Block Disabled:      (Update License Status)  
Suspended or Unlicensed Members Cannot View Code.

and


Block Disabled:      (Update License Status)  
Suspended or Unlicensed Members Cannot View Code.


RichieBoy67 12 Aug 2016 23:11

I think it is supposed to be $badPasswords

kerrghann 13 Aug 2016 08:36

Nope. I set my array to be $badPassword[0] - $badPassword[24]


All times are GMT. The time now is 12:59.

Powered by vBulletin® Version 3.8.14
Copyright © 2022, MH Sub I, LLC dba vBulletin. All Rights Reserved. vBulletin® is a registered trademark of MH Sub I, LLC
Copyright ©2001 - , vbulletin.org. All rights reserved.