Skip to content

Evita NullPointerException al buscar keystore Firefox en algunos casos en Linux#313

Open
albfernandez wants to merge 2 commits intoctt-gob-es:masterfrom
albfernandez:npe-MozillaKeyStoreUtilitiesUnix
Open

Evita NullPointerException al buscar keystore Firefox en algunos casos en Linux#313
albfernandez wants to merge 2 commits intoctt-gob-es:masterfrom
albfernandez:npe-MozillaKeyStoreUtilitiesUnix

Conversation

@albfernandez
Copy link
Copy Markdown
Contributor

@albfernandez albfernandez commented Mar 3, 2023

Error a resolver

NullPointerException en MozillaKeyStoreUtilitiesUnix.searchLastFirefoxVersion en algunos casos.

Resumen

Si intentamos buscar la versión de Firefox en un directorio que no tenemos permisos pero si que vemos, entonces da un NullPointerException, ya que el método isDirectory nos devuelve true pero el método list nos devuelve null, y se está suponiendo erróneamente que si isDirectory devuelve true, list devolverá un array

if (directoryList.isDirectory() {
    // ...
    for (final String filename : directoryLib.list()) { // NPE aqui porque list() devuelve null
        // ...
    }
}

Relacionado con #252

Detalles

Tenemos un directorio de otro usuario, del que no tenemos permisos, ejemplo:

drwx------  2 root    root    4,0K mar  3 01:02 prueba

Si ejecutamos un programa con otro usuario que no sea root, el método

file.isDirectory();

nos devuelve true, porque el directorio existe.

Pero al ejecutar

file.list();

Nos devuelve null.

Si bien la documentación de Java nos puede dar lugar a confusión, puesto que da a entender lo que pone el código actual: que si isDirectory()devuelve true, list() no devolverá null. Sin embargo en este caso especial no es así, y es necesario poner la comprobación de nulo propuesta en esta Pull Request.

If this abstract pathname does not denote a directory, then this method returns null. 
Otherwise an array of strings is returned, one for each file or directory in the directory.

traza en la versión 1.7.1

Exception in thread "main" java.lang.ExceptionInInitializerError
at es.gob.afirma.keystores.mozilla.MozillaKeyStoreUtilities.getSystemNSSLibDir(MozillaKeyStoreUtilities.java:251)
at es.gob.afirma.keystores.mozilla.MozillaKeyStoreUtilities.loadNSS(MozillaKeyStoreUtilities.java:694)
at es.gob.afirma.keystores.mozilla.NssKeyStoreManager.getNssProvider(NssKeyStoreManager.java:122)
at es.gob.afirma.keystores.mozilla.NssKeyStoreManager.init(NssKeyStoreManager.java:59)
at es.gob.afirma.keystores.mozilla.MozillaUnifiedKeyStoreManager.init(MozillaUnifiedKeyStoreManager.java:75)
at es.gob.afirma.keystores.AOKeyStoreManagerFactory.getNssKeyStoreManager(AOKeyStoreManagerFactory.java:511)
at es.gob.afirma.keystores.AOKeyStoreManagerFactory.getMozillaUnifiedKeyStoreManager(AOKeyStoreManagerFactory.java:542)
at es.gob.afirma.keystores.AOKeyStoreManagerFactory.getAOKeyStoreManager(AOKeyStoreManagerFactory.java:133)
at es.gob.afirma.standalone.SimpleAfirma.main(SimpleAfirma.java:708)
Caused by: java.lang.NullPointerException
at es.gob.afirma.keystores.mozilla.MozillaKeyStoreUtilitiesUnix.searchLastFirefoxVersion(MozillaKeyStoreUtilitiesUnix.java:145)
at es.gob.afirma.keystores.mozilla.MozillaKeyStoreUtilitiesUnix.getNssPaths(MozillaKeyStoreUtilitiesUnix.java:78)
at es.gob.afirma.keystores.mozilla.MozillaKeyStoreUtilitiesUnix.(MozillaKeyStoreUtilitiesUnix.java:26)
... 9 more

@albfernandez
Evita NPE al listar directorio.
30f0763 
Si intentamos buscar la version de firefox en un directorio
que no tenemos permisos pero si que vemos, entonces
da un NPE, ya que el metodo isDirectory nos devuelve true
pero el metodo list nos devuelve null, y se está suponiendo
erroneamente que si isDirectory devuelve true
list devolverá un array

Relacionado con ctt-gob-es#252
@clawgrip
Copy link
Copy Markdown
Contributor

clawgrip commented Mar 3, 2023 via email

@albfernandez
Copy link
Copy Markdown
Contributor Author

albfernandez commented Mar 3, 2023 •
edited
Loading

Hola
Acabo de probarlo y si, con canRead valdría, pero si lo ponemos así mejor poner canExecute también:
con canRead podríamos listar los subdirectorios pero si no tenemos canExecute nos devolverá una lista de sub-directorios a los que no podemos acceder.

if (directoryLib.isDirectory() && directoryLib.canRead() && directoryLib.canExecute()){
    // ..
}

@clawgrip
Copy link
Copy Markdown
Contributor

clawgrip commented Mar 3, 2023 via email

@albfernandez
Copy link
Copy Markdown
Contributor Author

albfernandez commented Mar 6, 2023

Actualizada la Pull Request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@albfernandez @clawgrip