Complete.Org: Mailing Lists: Archives: freeciv-dev: May 2005:
[Freeciv-Dev] (PR#13178) Major bug allows cheating with techpenalty=0
Home

[Freeciv-Dev] (PR#13178) Major bug allows cheating with techpenalty=0

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: lists@xxxxxxxxxxx
Subject: [Freeciv-Dev] (PR#13178) Major bug allows cheating with techpenalty=0
From: "Mateusz Stefek" <mstefek@xxxxxxxxx>
Date: Thu, 26 May 2005 06:16:14 -0700
Reply-to: bugs@xxxxxxxxxxx

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=13178 >

> [lists@xxxxxxxxxxx - Thu May 26 10:40:25 2005]:

Good report

> Ok, since i had problems with a cheater yesterday (yes mrvn, thats you
> :) i
> tried to pinpoint the method of cheating (the bulbnumbers are wrong,
> doesnt
> matter)
> 
> Bug is in Line 631 in /server/plrhand.c
> The condition to give Bulbs back is:
>  } else if (tech == plr->research.changed_from) {
> 
> But it should be:
> } else if (tech == plr->research.changed_from && !plr->got_tech) {
> 
Or better just reset plr->research.changed_from every time a tech is 
found. Patch attached

> P.S.: Do i get an entry in changelog as elfstone? That would be nice
> :)
Ok you will be noticed in cvs changelog as elfstone, however I don't 
know how it will be converted into Changelog file.
--
mateusz

Attachment: 13178.diff
Description: Binary data


[Prev in Thread] Current Thread [Next in Thread]