Good point. The proper way to do it would be to change the code above to:
Code:
setcookie("bbuserid",$userinfo[userid],0,$cookiepath);
setcookie("bbpassword",substr(md5($userinfo[password]),0,strlen($userinfo[password])),0,$cookiepath);
setcookie("bbuserid",$userinfo[userid],mktime(0,0,0,0,0,2020),$cookiepath);
setcookie("bbpassword",substr(md5($userinfo[password]),0,strlen($userinfo[password])),mktime(0,0,0,0,0,2020),$cookiepath);
Then in your browser (IE does this), DISABLE permanent cookies and ENABLE temporary ones.:
At the top menu: Tools, Internet Options> Security, Custom Level
Scroll down and with the two cookie settings, disable the first and enable the second.
The above code change will first set a temporary cookie for everyone then will attempt to overwrite it with a permanent one. If you don't allow permanent ones, you will simply keep the temporary and therefore logout when closing the browser.