Age | Commit message (Collapse) | Author | Files | Lines |
|
Prolific use of immediate constants strikes again.
Side note: why are these limits not in clientdata?
****
mana/plus!168
|
|
2024-08/01/#Debug.log:[14:25:43] Wrong actual or planned inbound packet
size!. Packet id: 205(0xcd), Planned size: 3, Actual size: 6
2 bytes packet id + 1 byte flag = 3
2 bytes packet id + 4 byte flag (wrong) = 6
Squashed with:
* Separate processKickAck handler for eathena
* Move ea's adminrecv and its sole handler processKickAck into TMWA
namespace
* Revert "Fix wrong read of clif_GM_kickack packet"
This would break TMWA packet processing, as @thorbjorn pointed out.
This reverts commit efe43bac4de90b3e427d985b77cb3dd3206bcf9d.
****
mana/plus!148
|
|
****
mana/plus!147
|
|
Not sure how to trigger this one, couldn't do it with a few simple
tests.
****
mana/plus!149
|
|
2025-03/22/#Debug.log-[19:38:37] Wrong actual or planned inbound packet size!. Packet id: 433(0x1b1), Planned size: 7, Actual size: 4
2025-03/22/#Debug.log:[19:38:52] You picked up 1 [@@760|Overlord's Helmet@@].
****
mana/plus!146
|
|
Pointless change, I know.
****
mana/plus!115
|
|
****
mana/plus!114
|
|
use @toevent, then switch character (and stay on character screen).
Thread 1 "manaplus" received signal SIGSEGV, Segmentation fault.
0x.. in Game::getCurrentMap (this=<optimized out>) at ./game.h:108
108 { return mCurrentMap; }
(gdb) bt
0x.. in Game::getCurrentMap (this=<optimized out>) at ./game.h:108
Ea::PlayerRecv::processMapMask (msg=...) at net/ea/playerrecv.cpp:242
****
mana/plus!108
|
|
Squashed with:
* Appease linters + more reformat
* Further improve skill error handling and messages.
24751878 "Improve skill error messages." added trailing dots only on the
second branch. This adds missing dots on the first one as well as
compacts the logic.
****
mana/plus!93
|
|
& shaves ~5KB from debug binary.
previd=""; while read -r line; do if [[ $line =~ .*"msg.read"[^\"]*"\""([^\"]*).* ]]; then id="${BASH_REMATCH[1]}"; if [[ "$id" == "$previd" ]]; then printf "%s\n" "$line"; fi; previd="$id"; fi; done < <(grep -r 'msg.read.*' src/net/)
****
mana/plus!91
|
|
How to reproduce: spam emote then quickly switch char
Thread 1 "manaplus" received signal SIGSEGV, Segmentation fault.
SkillDialog::getSkill (this=0x0, id=id@entry=1) at gui/windows/skilldialog.cpp:805
0 SkillDialog::getSkill (this=0x0, id=id@entry=1) at gui/windows/skilldialog.cpp:805
1 EAthena::SkillRecv::processSkillFailed (msg=...) at net/eathena/skillrecv.cpp:302
...
****
mana/plus!92
|
|
This has the side effect that now #Debug history is also loaded.
****
mana/plus!84
|
|
What happened:
1) herc evol sends 0x100 as packet ID offset
2) 0x100 is added to 0x0062 of SMSG_CHAR_PASSWORD_RESPONSE
3) SMSG_CHAR_PASSWORD_RESPONSE is then inserted into 0x162
slot of packet map, shadowing SMSG_GUILD_SKILL_INFO
4) on guild lvl-up, server sends this in SMSG_GUILD_SKILL_INFO
5) ManaVerse exits in confusion.
Note 1: due to popupmenu paste not working in change password dialog, tested password change only with invalid old password, which worked (opened error dialog instead of client exiting).
Note 2: Some packet handlers are bound using &Ea::LoginRecv prefix, this one had &LoginRecv and this was kept this way (uses vtable lookup?).
~~TODO: SMSG_CHAR_PASSWORD_RESPONSE seems to be something about password changes, but I could not find this packet either in TMWA or herc. Still, I need to test if these still work on all active servers~~ Tested.
Squashed with:
* Fix exit on password change, too
* Fix guild lvl-up packet parsing causing client exit in ML
****
mana/plus!81
|
|
|
|
|
|
See #73 for discussion
Squashed with:
* Fix mana pearl showing in arrows slot in Classic
* Fix off-by-one. Was causing Old Towel to render in torso slot in ML.
****
mana/plus!78
|
|
Other addButton sections intentionally left misaligned as a test for future fix in nsiqcppstyle.
.. why does it not complain about every such block?
|
|
See merge request mana/plus!64
|
|
No server in use still sends that field.
****
Approved-by: Jesusalva Jesusalva <jesusalva@themanaworld.org>
|
|
****
Approved-by: Jesusalva Jesusalva <jesusalva@themanaworld.org>
Approved-by: Thorbjørn Lindeijer <bjorn@lindeijer.nl>
|
|
Co-Authored-By: jak1 <jak1@themanaworld.org>
Co-Authored-By: jesusalva <jesusalva@themanaworld.org>
Co-Authored-By: Fedja Beader <fedja@protonmail.ch>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Also drop support for 65535 item id on old packet versions.
|
|
|