vulnerabilidad de código malicioso - puede exponer representación interna mediante la devolución de referencia a objeto mutable

Question

Hola estoy recibiendo la violación de la siguiente manera:

vulnerabilidad de código malicioso - puede exponer representación interna por regresar referencia al objeto mutable

en mi código que he escrito como éste

public String[] chkBox() { 
    return chkBox; 
} 

Cómo podemos resolverlo.

Answer 1

Como dice el mensaje de error, usted está volviendo estado interno (chkBox es - muy probablemente - parte del estado interno de un objeto a pesar de que usted no está mostrando su definición)

Esto puede causar problemas si que - por ejemplo - hace

String[] box = obj.chkBox(); 
box[0] = null; 

Puesto que un objeto de matriz, ya que todos los objetos de Java, se pasa por referencia, esto va a cambiar la matriz original almacenado dentro de su objeto, así.

Lo más probable es que desea hacer para solucionar este problema es un simple

return (String[])chkBox.clone(); 

que devuelve una copia de la matriz en lugar de la matriz real.

Answer 2

Supongamos lo siguiente:

1. Su clase hace algo que importa desde una perspectiva de seguridad o privacidad, y que el estado de chkbox se utiliza de alguna manera en la implementación clases de sus mecanismos de privacidad/seguridad.

2. El método chkBox() se puede invocar mediante algún código que no sea de confianza.

Consideremos ahora este código:

// ... in an untrusted method ... 

Foo foo = ... 
String[] mwahaha = foo.chkBox(); 
mwahaha[0] = "Gotcha!"; // ... this changes the effective state of `Foo` 

Al devolver una referencia a la matriz real que representa la chkbox, usted está permitiendo que el código externo a la clase Foo a meter la mano y cambiar su estado.

Esto es malo desde una perspectiva de diseño (se llama "abstracción con fugas"). Sin embargo, si esta clase se utiliza en un contexto donde también puede haber un código que no es de confianza, este (el método chkBox()) es un agujero de seguridad potencial. Eso es lo que te dice el mensaje de infracción.

(Por supuesto, el corrector de código no tiene manera de saber si esta clase en particular es realmente de seguridad crítica. Eso es para que usted pueda entender. Lo que en realidad está diciendo es "Hey! Mira! Este es sospechoso!")

---

La solución depende de si el código (o, de hecho toda la biblioteca o aplicación) es crítico de seguridad ... o código de seguridad crítica sea en un futuro despliegue. Si se trata de una falsa alarma que, Sólo pudo reprimir la violación; es decir, se marca de manera que serán ignorados por el comprobador de Si esto es un problema real (o podría convertirse en un problema real), entonces o bien devolver una copia de la matriz:.

return (String[]) chkBox.clone(); 

Pero claramente, hay un costo de rendimiento en la clonación de la matriz cada vez que llame al chkBox. Alternativamente, podría modificar el chkBox método para devolver un elemento seleccionado de la matriz:

public String chkBox(int i) { 
     return chkBox[i]; 
    } 

En este caso, sospecho que el enfoque alternativo será mejor ... aunque depende de cómo se utiliza actualmente el método.