search

zhyrohaad / AlasiyaEvE

Public repo for bug tracking the private Alasiya-EvE codebase
2 stars 0 forks source link

Docking up while being target painted causes a segfault #108

Open ryllae opened 4 years ago

ryllae commented 4 years ago

Equip a fleet republic target painter, activate on a player character in high sec. Have the other toon dock up. SEGFAULT.

zhyrohaad commented 4 years ago

that tells me that module code thats supposed to be doing "ship status" updates (jump, dock, warp, etc) are NOT processing correctly. i'll look into this tonite.

edit it's ANY module activated on a docking ship, not just painters edit again and its not just docking. my status update processor was faulty and incomplete. did some more work to it. test again and report when ya get time, pls.

ryllae commented 4 years ago

Haven't tested target painting, but rats acquiring a lock while docking segfaults. https://github.com/zhyrohaad/AlasiyaEvE/issues/75 Related?

zhyrohaad commented 4 years ago

Not really. TargMgr is separate system. However, never expected rats targeting while docking, so never tested that scenario.

ryllae commented 4 years ago

Rats could be shooting you while warping or session changing, so I assume you'd have the same problem there either way. It's just easier for me to test docking.

zhyrohaad commented 4 years ago

I hadn't thought of it, but you're right. However it's not coded that way, in contrast to my generic module "target destroyed" call. The dock, jump,warp, etc, is specific, but should not be. This highlights a shortcoming in the code and need to be addressed. This and the referenced module code are top priority.

zhyrohaad commented 4 years ago

This has been updated and should work for all possible scenarios

EDIT - UPDATE after digging into this a bit more, i have found TargetManager code is old, outdated, and about half of it's responses to client are being ignored, as it was written for the Apoc version. am currently rewriting functions and methods, and updating responses.

ryllae commented 4 years ago

I think your latest comment confirmed this, but it's still broken.

zhyrohaad commented 4 years ago

target manager has been updated, and seems to be working correctly from my testing. update pushed to main server.

ryllae commented 4 years ago

Nope. I undocked a Hawk and a Noctis. Had the Hawk target paint the Noctis. Crashes when the Noctis docks.

zhyrohaad commented 4 years ago

touch

ryllae commented 4 years ago

Working correctly on dev eb777b.

zhyrohaad commented 4 years ago

main server updated

ryllae commented 4 years ago

Still crashes.

zhyrohaad commented 4 years ago

must be something funny on main server, as i cannot reproduce this on dev

ryllae commented 3 years ago

Still crashes.