Issue information

Issue ID
#7988
Status
Fixed
Severity
None
Started
Lilori
Jan 23, 2014 11:08
Last Post
Angelmelody
Mar 8, 2014 18:42
Confirmation
Yes (2)
No (1)

Lilori - Jan 23, 2014 11:08

[u][b]Use Last GIT of Hercules[/b][/u]

Reproduce:[list=1]
[*]Use your royal guard
[*]Use your skill Exceed Break
[*]Map server crashed successfuly.
[/list]

Mhalicot - Jan 23, 2014 11:51

[color=#009acd][font='trebuchet ms', helvetica, sans-serif]Using Clean last Git revision, Failed to reproduce, it ddnt. crashed after using Exceed Break 2x..[/font][/color]

[color=#009acd][font='trebuchet ms', helvetica, sans-serif]Maybe there something in your src modification that affects your map server causing it to crashed.[/font][/color]
[color=#009acd][font='trebuchet ms', helvetica, sans-serif][img]http://i41.tinypic.com/2805bg0.jpg[/img][/font][/color]

Lilori - Jan 23, 2014 12:47

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

pan - Jan 23, 2014 16:10

Couldn't reproduce either

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

This post has been edited by pan on Jan 27, 2014 3:38

Mhalicot - Jan 23, 2014 23:24

[color=#009acd][font='trebuchet ms', helvetica, sans-serif]pleas run [url="http://herc.ws/wiki/MSVC_Crash_Debugging"]http://herc.ws/wiki/MSVC_Crash_Debugging[/url][/font][/color]

malufett - Jan 25, 2014 13:29

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

:meow:

pan - Jan 27, 2014 4:07

Open db/re/skill_require_db.txt and find:[code=auto:0] 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# [/code]Replace it with:[code=auto:0] 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# [/code]Also, open your src/map/skill.c and find:[code=auto:0] 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; [/code]Replace it with:[code=auto:0] 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; [/code]

This post has been edited by pan on Jan 27, 2014 4:09

Mhalicot - Jan 27, 2014 4:15

[color=#009acd][font='trebuchet ms', helvetica, sans-serif]^ its nice if your going to make a pull request for that. :meow:[/font][/color]

pan - Jan 27, 2014 4:21

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:
[url="http://pastebin.com/nvZmBkYh"]http://pastebin.com/nvZmBkYh[/url]



EDIT:
Already made pull request, but somehow it merged with my last one...
[url="https://github.com/HerculesWS/Hercules/pull/254"]https://github.com/HerculesWS/Hercules/pull/254[/url]

This post has been edited by pan on Jan 27, 2014 4:31

Mhalicot - Jan 27, 2014 4:45

[color=#009acd][font='trebuchet ms', helvetica, sans-serif]^ Haha thats nice, lets wait for devs to take a look at it.[/font][/color]

malufett - Feb 10, 2014 10:14

Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414

Thanks pan..

:meow:

HisokaMachi - Mar 8, 2014 18:14

[quote name="malufett" timestamp="1392027282"]
Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414

Thanks pan..

:meow:[/quote]

can you link it again where i can see the fix

Angelmelody - Mar 8, 2014 18:42

[quote name="HisokaMachi" timestamp="1394302455"][quote name="malufett" timestamp="1392027282"]

Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414

Thanks pan..

:meow:[/quote]
can you link it again where i can see the fix[/quote]

[url="https://github.com/HerculesWS/Hercules/commit/9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414"]https://github.com/HerculesWS/Hercules/commit/9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414[/url]