[morefarming] Wielded item et crash serveur #62

New Issue

sys4 · 2020-08-29T21:44:30+02:00

sys4 commented

2020-08-29 21:44:30 +02:00

Bug remonté par Grossam provenant de ce ticket : nalc/maidroid#2

Il existe une façon de faire crasher le serveur quand le mod maidroid est chargé avec le mod farming_redo et que le core farming provement du mod morefarming est activé.

Si nous retirons l'item qui se trouve dans la case wielded item du formspec du maidroid avec le core farming activé, un crash serveur peut survenir si cet item est entrain d'être planté par le maidroid.

C'est en réalité la fonction place_seed() du mod farming, croyant avoir à faire à une graine, qui fait crasher le serveur car la graine ayant été retiré par le joueur, la référence de l'item devient nulle.

Donc il manque surement une vérification à faire qqpart.

De plus il subsiste un bug quand la pile d’item (le stack) est dans la main du maidroid (wielded item) et qu’il cherche à le mettre en terre, il n’en plante en réalité qu’un seul mais la pile disparait totalement ce qui fait perdre un item.

Bug remonté par Grossam provenant de ce ticket : https://sys4.fr/gitea/nalc/maidroid/issues/2 Il existe une façon de faire crasher le serveur quand le mod maidroid est chargé avec le mod farming_redo et que le core farming provement du mod morefarming est activé. Si nous retirons l'item qui se trouve dans la case wielded item du formspec du maidroid avec le core farming activé, un crash serveur peut survenir si cet item est entrain d'être planté par le maidroid. C'est en réalité la fonction place_seed() du mod farming, croyant avoir à faire à une graine, qui fait crasher le serveur car la graine ayant été retiré par le joueur, la référence de l'item devient nulle. Donc il manque surement une vérification à faire qqpart. De plus il subsiste un bug quand la pile d’item (le stack) est dans la main du maidroid (wielded item) et qu’il cherche à le mettre en terre, il n’en plante en réalité qu’un seul mais la pile disparait totalement ce qui fait perdre un item.

sys4 added this to the NALC-1.3.3 milestone 2020-08-29 21:44:30 +02:00

sys4 added the

BUG

Approuvé

labels 2020-08-29 21:44:30 +02:00

sys4 self-assigned this 2020-08-29 21:44:31 +02:00

sys4 changed title from ~~[maidroid] Wielded item et crash serveur~~ to [morefarming] Wielded item et crash serveur

2020-08-29 22:01:02 +02:00

sys4 referenced this issue from a commit

2020-08-29 22:57:46 +02:00

(#62)[morefarming] Corrige crash et count wielded item maidroid

sys4 commented