Skip to content

fix: crash getItems#892

Merged
luanluciano93 merged 4 commits intoopentibiabr:mainfrom
luanluciano93:fix-getItems
Oct 5, 2024
Merged

fix: crash getItems#892
luanluciano93 merged 4 commits intoopentibiabr:mainfrom
luanluciano93:fix-getItems

Conversation

@luanluciano93
Copy link
Copy Markdown
Contributor

image

@bateunatrave
Copy link
Copy Markdown

Now the trade window is appearing, but when the other player's item appears, the crash occurs.

@luanluciano93
Copy link
Copy Markdown
Contributor Author

Now the trade window is appearing, but when the other player's item appears, the crash occurs.

Can you use the deputy to see where the crash is occurring, like the PR image?

@kokekanon
Copy link
Copy Markdown
Contributor

kokekanon commented Oct 2, 2024

test in 13.40

in my test there is no crash, but when I accept the trade

i don't know if the error is in canary or otc. but for some reason
server sent 0x89 to client.

ERROR: ProtocolGame parse message exception (357 bytes, 348 unread, last opcode is 0x89 (137), prev opcode is 0xffffffff (-1)): unhandled opcode 137
Packet has been saved to packet.log, you can use it to find what was wrong. (Protocol: 1340)
src/client/protocolcodes.h
GameServerCloseForgeWindow = 137,


src/client/protocolgame.h
void parseCloseForgeWindow(const InputMessagePtr& msg);


src/client/protocolgameparse.cpp
....
                case Proto::GameServerCloseForgeWindow:
                    parseCloseForgeWindow(msg);
                    break;
                    
...
                    
void ProtocolGame::parseCloseForgeWindow(const InputMessagePtr& msg)
{
    g_lua.callGlobalField("g_game", "onCloseForgeWindow"); // TODO: implement 
}

in 8.6 this error does not occur

jordanhoare added a commit to tibia-oce/otclient that referenced this pull request Oct 11, 2024
* perf: Drawpool 3.1

more stability in FPS

* setting m_effectAlpha, m_missileAlpha to 1.f

* fix: display density on mapview

* feat: hud scale

* fix: Interface Anchors (opentibiabr#862)

* feat: support directx

* improve: use egl v3

* small change in gc

* fix: memory leak in controller

* improve: organization of the project structure

* add new config to draw covered things

There are projects where this system doesn't work very well, so if your project is one of those, just set it to true to fix the tiles that aren't being drawn.

* feat: update background image and startup sound to use 13.40 cip files. Adjustments in options buttons order and logout button icon. (opentibiabr#865)

* update UIGraph (need test)

* fix shader warning

* fix: console error due to shaders

* draw-covered-things to false

Note: Setting false will not draw objects that the client understands are covered. If for some reason a tile is not being drawn because your game is custom, set this property to true and this issue will be resolved.

* baseline updated (update your vcpkg)

git pull

* Revert "baseline updated (update your vcpkg)"

This reverts commit 8dda995.

* fix: opentibiabr#867

* feat: extractTexture in framebuffer, Arch SS2 by default and cleanup

* fix: inverted texture in extractTexture

* html: support tag img

* perf: reducing locking, thus increasing performance a little.

* fix draw shader

* prevent crash in static text

* improve: add lua call protection

* perf: improve propagateOnMouseMove

* fix: bot item ondrop and analyzer loot channel (opentibiabr#887)

* fix: outfit size , prevent ERROR: invalid thing (opentibiabr#884)

Co-authored-by: Rodrigo Paixão <god.rodrigo@hotmail.com>

* improve: HTTP login error handling to display login.php error messages (opentibiabr#886)

* fix: compilation error on some pcs

* feat: full cyclopedia module (opentibiabr#868)

* fix: game_store bugs. fix: opentibiabr#873 (opentibiabr#874)

* feat: Minimap GM teleport ( ctrl +shift + click on minimap) (opentibiabr#885)

Co-authored-by: Luan Luciano <luanluciano@outlook.com>

* support "Mage" vocation (opentibiabr#882)

several OT servers merge MS/ED vocation into a Mage vocation having the best features of both MS and ED.

Without this fix, on those servers, vBot would constantly error out with Error: (...) concatenat voc: a nil value

* fix (opentibiabr#900)

* improve parseOpenContainer and fix error with decoration kit (opentibiabr#890)

* fix: optional display rarity colors in items . feature GameColorizedLootValue 12.00 (opentibiabr#896)

* fix: crash getItems (opentibiabr#892)

* looting blacklist (opentibiabr#881)

When "loot every item" is off, it will loot everything in the loot list.

When "loot every item" is on, it will loot everything that is not in the loot list, effectively making the loot list a blacklist when "loot every item" is on.

* Fix black and red squares behaviors (opentibiabr#902)


* Fix squares:
- Set black squares color to 1 instead of 0 so it's not treated as alpha
- Fix creaturesMark logic so red squares don't disappear when attacking and black squares show when attacked

* feat: display tier UI (GameThingUpgradeClassification) (opentibiabr#895)

* feat: Vips Groups | fix issue: opentibiabr#730 (opentibiabr#908)

* fix: spacing for align bottom vertical layouts (opentibiabr#909)

* Fix: opentibiabr#901

---------

Co-authored-by: Renato Machado <mehahx@gmail.com>
Co-authored-by: SkullzOTS <86809689+SkullzOTS@users.noreply.github.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Felipe Muniz <felipe.silva@virtus.ufcg.edu.br>
Co-authored-by: Rodrigo Paixão <god.rodrigo@hotmail.com>
Co-authored-by: kokekanon <114332266+kokekanon@users.noreply.github.com>
Co-authored-by: divinity76 <divinity76@gmail.com>
Co-authored-by: divinity76 <hans@loltek.net>
Co-authored-by: OT Archive <112215823+OTArchive@users.noreply.github.com>
Co-authored-by: Marcin Jałocha <nekiro.dev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants