Jump to content

  •  

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

  • #007988

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (2)No (1)
Photo

[Royal Guard] Exceed Break Causes Map Server Crash

Posted by Lilori on 23 January 2014 - 11:08 AM

Use Last GIT of Hercules

Reproduce:
  • Use your royal guard
  • Use your skill Exceed Break
  • Map server crashed successfuly.

Using Clean last Git revision, Failed to reproduce, it ddnt. crashed after using Exceed Break 2x..

Maybe there something in your src modification that affects your map server causing it to crashed.
Posted Image

changed status to: Unable To Reproduce

that's weird,
last 30 minutes, i did with clean GIT, and got map server crash. -,-

Couldn't reproduce either

EDIT:
Just checked following Malufett's instructions and could reproduce

Edited by pan, 27 January 2014 - 03:38 AM.


changed status to: Confirmed

it will crashed if you don't have weapons equipped...

:meow:

Open db/re/skill_require_db.txt and find:
2316,0,0,24:28:32:36:40,0,0,0,99,0,0,none,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 //LG_EXEEDBREAK#Exceed Break#
Replace it with:
2316,0,0,24:28:32:36:40,0,0,0,2:3:4:5,0,0,none,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 //LG_EXEEDBREAK#Exceed Break#
Also, open your src/map/skill.c and find:
			case SC_EXEEDBREAK:
				val1 *= 150; // 150 * skill_lv
				if( sd && sd->inventory_data[sd->equip_index[EQI_HAND_R]] ) { // Chars.
					val1 += (sd->inventory_data[sd->equip_index[EQI_HAND_R]]->weight/10 * sd->inventory_data[sd->equip_index[EQI_HAND_R]]->wlv * status->get_lv(bl) / 100);
					val1 += 15 * (sd ? sd->status.job_level:50) + 100;
				} else // Mobs
					val1 += (400 * status->get_lv(bl) / 100) + (15 * (status->get_lv(bl) / 2)); // About 1138% at mob_lvl 99. Is an aproximation to a standard weapon. [pakpil]
				break;
Replace it with:
			case SC_EXEEDBREAK:
				val1 *= 150; // 150 * skill_lv
				if( sd )
				{
					if( sd->equip_index[EQI_HAND_R] >= 0 && sd->inventory_data[sd->equip_index[EQI_HAND_R]] )
					{
						val1 += (sd->inventory_data[sd->equip_index[EQI_HAND_R]]->weight/10 * sd->inventory_data[sd->equip_index[EQI_HAND_R]]->wlv * status->get_lv(bl) / 100);
						val1 += 15 * (sd ? sd->status.job_level:50) + 100;
					}
				}
				else // Mobs
					val1 += (400 * status->get_lv(bl) / 100) + (15 * (status->get_lv(bl) / 2)); // About 1138% at mob_lvl 99. Is an aproximation to a standard weapon. [pakpil]
				break;

Edited by pan, 27 January 2014 - 04:09 AM.


^ its nice if your going to make a pull request for that. :meow:

Well, I have one pull request that's been in that list for a week now, I'll see if I can make one today c:
Also I've documented skill_require_db, figuring out how to edit that thing was a mess:
http://pastebin.com/nvZmBkYh



EDIT:
Already made pull request, but somehow it merged with my last one...
https://github.com/H...rcules/pull/254

Edited by pan, 27 January 2014 - 04:31 AM.


^ Haha thats nice, lets wait for devs to take a look at it.

changed status to: Fixed

Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414

Thanks pan..

:meow:

Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414

Thanks pan..

:meow:


can you link it again where i can see the fix



Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414

Thanks pan..

:meow:

can you link it again where i can see the fix


https://github.com/H...1f90bc4922bf414