[TUTORIAL][r40250] Bug-Fix for CheckClientVersion

01/03/2014 07:02 redrpci2008#1

Hello, dear community!

Since I've read lollo_9_1's topic ( [Only registered and activated users can see links. Click Here To Register...] ) about fixing the client version checking bug, I've thought that it would be useful that a fix for the source code would exist, since this is now used as a security measure.

Simply open the Srcs\Server\game\src\config.cpp file, go to line 1288,
and change this code:

Code:

if (version > date)

with this code:

Code:

if (version != date)

That's all! Happy compiling! :)

01/03/2014 07:13 IchVerabschiedeMich#2

You just use the wrong binary ;)

01/03/2014 07:16 redrpci2008#3

Quote:
Originally Posted by Padrio
why not just this:
Code:
if (true)
You just use the wrong binary ;)

I'm using mainline.

And since there are 2 integer values that must be compared, this is the solution for my code.

[Only registered and activated users can see links. Click Here To Register...]

01/03/2014 07:32 .PolluX#4

Why that mutch threads for source? One Big Thread for all of them like " Kleiner Releaeses " Would be batter than spamming this board with all of them..^^

01/03/2014 08:52 TheMarv :<#5

I'd recommend taking out the if instead of replacing > with !=

01/03/2014 09:01 redrpci2008#6

Quote:

Originally Posted by TheMarv :<

I'd recommend taking out the if instead of replacing > with !=

If I take out the if, the core just simply disconnects all players :awesome:

01/03/2014 09:07 TheMarv :<#7

Quote:

Originally Posted by redrpci2008

If I take out the if, the core just simply disconnects all players :awesome:

oh yeah just saw your screenshot :D
I thought the if-statement was for the opposite

07/15/2014 20:56 lollo_9_1#8

Note: The ClientVersion is checked thrice:

Code:

$ fgrep "version > date" *
config.cpp:		if (version != date) // @fixme103 (version > date)
input_login.cpp:				if (version != date) // @fixme103 (version > date)
input_main.cpp:					if (version != date) // @fixme103 (version > date)

07/15/2014 21:04 .T4Ump#9

Everyone has corrected source..