I can tell you right now it won't work.
param() is a CGI.pm function and thusly (since the OO method is being used) all references to param() need to be written as $q->param(). I see at least 6 instances where it is wrong.
Not to mention that:
open("USER",param(user).'.cgi') || die ("Can't Regester you: $! ");
is:
1) a security hole the size of Texas.
2) written incorrectly anyway. USER should not be in quotes, and it is preferable to use 'or' instead of '||' in this situation.
Plus, I see calls to subroutines called lock and unlock (lock being called AFTER information is read in from the file, which is pointless..), yet neither of them appear in the file, nor are any other files require()ed in.
Not recommended that you or anyone else put this on a live server.
--mark
This message has been edited by Mark Badolato on October 22, 2000 at 09:10 PM
