Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

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

wants to merge 2 commits into from

Conversation

albfernandez
Copy link
Contributor

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
Contributor

clawgrip commented Mar 3, 2023 via email

@albfernandez
Copy link
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
Contributor

clawgrip commented Mar 3, 2023 via email

@albfernandez
Copy link
Contributor Author

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