In quali situazioni il field.setAccessible (true) di Java fallirà?

Question

Ho una situazione in cui il codice di un utente sta lanciando un IllegalAccessException su un campo a cui si accede per riflessione. Appena prima di accedere al campo, viene chiamato setAccessible(true). Quindi, mi sembra che questo metodo stia silenziosamente fallendo.

In quali situazioni ciò accadrebbe? Questo potrebbe avere qualcosa a che fare con un responsabile della sicurezza?

Ecco il frammento di codice che causa l'eccezione:

private static Field levelField; 
public int getLevel() { 
    try { 
     if (levelField == null) { 
      levelField = MessageInfo.class.getDeclaredField("level"); 
      levelField.setAccessible(true); 
     } 
     return levelField.getInt(this); // <-- IllegalAccessException thrown here 
    } catch (Exception e) { 
     handleException(e); 
    } 
    return ICompilationUnit.NO_AST; 
} 

Answer 1

setAccessible è documentato per lanciare un SecurityException. Si noti che la documentazione fornisce casi in cui SecurityException verrà lanciato anche se non è presente SecurityManager. Naturalmente potrebbe anche non riuscire a causa di un'eccezione asincrona: Thread.stop, eccezione relativa al buffer NIO o un errore JVM.

Il vero problema con questo codice (oltre a quello che utilizza la riflessione) è che c'è un campo che può essere impostato per essere parzialmente inizializzato. Ciò causa una condizione di competizione (hai una statica mutabile, quindi devi preoccuparti dei thread (suggerimento, evitare statiche mutabili!)).Un altro thread può chiamare getInt sullo stesso Field prima che venga chiamato setAccessible. E come sembra essere stato scoperto dall'interlocutore originale, non è nemmeno un'eccezione. Sarebbe molto più sicuro e più chiaro impostare il campo in un inizializzatore statico.

Answer 2

Da Java propria documentation per setAccessible():

Un SecurityException viene generato se flag è vero, ma l'accessibilità di uno degli elementi dell'array di input non può essere modificato (ad esempio, se l'oggetto elemento è un oggetto Constructor per la classe Class). Nel caso di tale SecurityException, l'accessibilità degli oggetti è impostata per contrassegnare gli elementi dell'array fino a (ed escludere) l'elemento per il quale si è verificata l'eccezione; l'accessibilità degli elementi oltre (e compresi) l'elemento per il quale si è verificata l'eccezione è rimasta invariata.

Purtroppo non menzionano nulla di un IllegalAccessException.

Answer 3

Non dovrebbe essere un problema di security manager - si otterrebbe un SecurityException o sottoclasse.

Il codice levelField.getInt(*this*) non sembra giusto ...

si dovrebbe essere passando un'istanza MessageInfo come parametro.

Stai chiamando questo all'interno della classe MessageInfo? (perché?!?) o una sottoclasse di MessageInfo? (Cercando di fare un campo privato di un atto superclasse come se fosse protetto? Vuol MessageInfo hanno un metodo di getLevel()? Se è così, si potrebbe chiamare super.getLevel() per ottenere il valore invece di tentare in questo modo.)

Se non è MessageInfo o una sottoclasse, questo è il tuo problema: hai il campo level della classe MessageInfo e stai cercando di ottenere il valore di quel campo dalla classe corrente. Anche se questo dovrebbe essere gettando un IllegalArgumentExeception invece di IllegalAccessException ...

Se è davvero 'IllegalAccessExeception' - prova a mettere un po 'di registrazione dentro quella if (levelField == null) blocca - assicurarsi che sia davvero di essere exececuted. Il campo è statico: potrebbero esserci altre istanze o metodi che impostano un valore su di esso.