Bug Tracker Migration

June 3rd
Good news everyone! The staff has decided that it is time to slowly kill off this Bug Tracker. We will begin the process of slowly migrating from this Bug Tracker over to our Github Issues which can be found here: https://github.com/HerculesWS/Hercules/issues

Over the next couple of days, I will be closing off any opportunity to create new reports. However, I still will keep the opportunity to reply to existing Bug Reports. Doing this will allow us to slowly fix any bug reports we have listed here so that we can easily migrate over to our Issue Tracker.

Update - June 7th 2015: Creating new bug posts has been disabled. Please use our https://github.com/HerculesWS/Hercules/issues tracker to post bugs. Users are still able to reply to existing bug posts.

- Administration

Issue Information

• Issue ID

  #008082

• Severity

  1 - Low

• Status

  Fixed

• Return to Issue Listing

Issue Confirmations

• Yes (2)No (0)

1

itemheal kills with high rand value

Posted by evilpuncker on 13 March 2014 - 06:29 AM

itemheal rand(2000000,3000000),rand(2000000,3000000);

use this and your character will be killed, I don't know if it is intended, but I really want it to work instead of killing the character

• Back to top

AnnieRuru

Mar 13 2014 06:47 AM

use heal instead of itemheal ?

heal rand(2000000,3000000),rand(2000000,3000000);

but its still a bug ...
click 'Yes'

• Back to top

pan

Mar 13 2014 06:50 PM

@AnnieRuru heal is quite different from itemheal, see my last commit

@evilpuncker
I couldn't reproduce this issue without trying to overflow, but even so I coded an overflow check... It's not a good idea to input values that are so near INT_MAX

Fixed in https://github.com/H...cd5e8c19e5d53f4

• Back to top

pan 

changed status to: Fixed, 

changed severity to: 1 - Low

evilpuncker

Mar 13 2014 07:01 PM

thanks pan

• Back to top

1. Bug Tracker
2. → Hercules Bug Archives
3. → Core
4. → Severity: Low
5. → Status: Fixed
6. Privacy Policy
7. Hercules Forum Rules ·