* Fix for "yet another online check bypass technique". (bugreport:2292)

git-svn-id: https://svn.code.sf.net/p/rathena/svn/trunk@13256 54d463be-8e91-2dee-dedb-b68131a5f0ec

Changelog-Trunk.txt

@@ -3,6 +3,8 @@ Date	Added
 AS OF SVN REV. 5091, WE ARE NOW USING TRUNK.  ALL UNTESTED BUGFIXES/FEATURES GO INTO TRUNK.
 IF YOU HAVE A WORKING AND TESTED BUGFIX PUT IT INTO STABLE AS WELL AS TRUNK.
 
+2008/10/03
+	* Fix for "yet another online check bypass technique". (bugreport:2292) [FlavioJS]
 2008/10/01
 	* Rev. 13252 Kaupe will now trigger from mobs, homuns and mercenaries, as well. (bugreport:2297) [L0ne_W0lf]
 	* Rev. 13251 Removed 'casttime' adjustment from offensive ressurection. Follow up to r13225. [L0ne_W0lf]

src/char/char.c

@@ -2295,10 +2295,11 @@ int parse_fromlogin(int fd)
 						WFIFOSET(i,3);
 						set_eof(i);
 					}
-					else //Shouldn't happen, but just in case.
+					else // still moving to the map-server
 						set_char_offline(-1, aid);
 				}
 			}
+			idb_remove(auth_db, aid);// reject auth attempts from map-server
 		}
 		break;
 		

src/char_sql/char.c

@@ -1904,10 +1904,11 @@ int parse_fromlogin(int fd)
 						WFIFOSET(i,3);
 						set_eof(i);
 					}
-					else //Shouldn't happen, but just in case.
+					else // still moving to the map-server
 						set_char_offline(-1, aid);
 				}
 			}
+			idb_remove(auth_db, aid);// reject auth attempts from map-server
 		}
 		break;