SuFile.isDirectory() can throw NoShellException (and shouldn't) about libsu HOT 4 CLOSED

I think this is expected behavior and it is the developer's responsibility to check that the shell is valid before using it.

from libsu.

I personally disagree that isDirectory() should throw NoShellException because SuFile extends File, when this issue crashed my app on someone device, I had no idea and it was a existing behavior for me, so this was very unexpected for me.

I already did my best do avoid calling SuFile when root does not exists or is not available, and this "behavior" was really unexpected, so I think it's definitely a flaw of the library because the goal of a library is to help developers make programs.

But this is mainly a mater of opinion, so I understand the disagreement.

from libsu.

Well, SuFile is backed by a shell; if you have no shell available, how does it operate? If you are really concerned about this, you should move over to use the nio module.

from libsu.

Well, I reported this because SuFile crashed despite my app checking for root access.

from libsu.