[BUG] FML : FMLSecurityManager

Mokona78

Si getClassContext(); renvoie un tableau de 4 ou 5 éléments ?

Superloup10

Aucune idée, je ne sais pas ce que renvoie ce getter, un tableau de class, mais lesquelles ?

Mokona78

Ça renvoie la callstack. Ce sont les différentes appels qui ont amenés à l’appel courant.

Mais peu importe:

un tableau de taille 4 (>3) auquel on accède à l’élément d’indice 4, ça fait IndexOutOfBoundsException.
un tableau de taille 5 (>4) auquel on accède à l’élément d’indice 5, ça fait pas mieux, pas pire.

Le test n’est pas cohérent avec l’appel qui suit.

Après, peut-être qu’il y a une règle qui dit que si le tableau renvoyé est assuré d’avoir une taille 4 alors il a une taille 5. Et s’il a une taille 5 il est assuré d’avoir une taille 6… J’y crois assez peu.

robin4002

http://www.minecraftforge.net/forum/index.php/topic,40907.msg215989.html#msg215989
“So anyway, I have to completly bypass the security manager using ASM.”
Retires cette phrase avant que lex ne la voit, sinon tu vas encore plus te faire allumer x)
Lex n’aime pas du tout les “comme vous n’acceptez pas mon correctif / ma fonctionnalité je vais tout hacker de mon côté avec de l’ASM”.

Mokona78

C’est pour un test runner, pas pour distribuer dans un mod. Je ne vois pas pourquoi il ne pourrait pas le faire.

De plus, il ajoute le crash log demandé qui montre bien que le problème est réel et pas hypothétique.

Superloup10

cpw a fini par merge le pull request. (voir https://github.com/MinecraftForge/MinecraftForge/pull/3125#issuecomment-236043532 )

RedRelay

@‘robin4002’:

http://www.minecraftforge.net/forum/index.php/topic,40907.msg215989.html#msg215989
“So anyway, I have to completly bypass the security manager using ASM.”
Retires cette phrase avant que lex ne la voit, sinon tu vas encore plus te faire allumer x)
Lex n’aime pas du tout les “comme vous n’acceptez pas mon correctif / ma fonctionnalité je vais tout hacker de mon côté avec de l’ASM”.

Non non, ce n’est pas du tout ça dont il est question x)
Indépendamment de ma volonté, je fais un System.exit() et ce n’est pas autorisé par le SecurityManager.
Donc la, bug ou pas, je dois bypass ^^
Mais ce n’est pas pour un mod, donc aucun soucis incompatibilité ^^

Mokona78

@‘Superloup10’:

cpw a fini par merge le pull request. (voir https://github.com/MinecraftForge/MinecraftForge/pull/3125#issuecomment-236043532 )

Ah ! Bien.

À noter que sa remarque sur le fait que cette classe s’active pendant une session de test est peut-être à revoir.

RedRelay

Bah c’est juste un System.exit() appelé par une classes externe au mod et à ses tests qui fait qu’on arrive dans cette fonction.
Je lance le jeu comme si c’était le launcher qui le lancé pour ne pas avoir a faire un new WorldServeur() manuellement par exemple et donc éviter d’avec un environnement de test différent de celui de production.
Mais oui, je suis d’accord que dans l’ideal, je n’ai pas a modifier le FMLSecurityManager, mais un mod n’a pas a appeler System.exit() ^^