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.

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

Channel.selectNext() is not fully compliance with OpenMobile 2.04 #75

Open
GoogleCodeExporter opened this issue Mar 26, 2015 · 0 comments

Comments

@GoogleCodeExporter
Copy link

according to Java Doc at 
http://seek-for-android.googlecode.com/svn/trunk/doc/org/simalliance/openmobilea
pi/Channel.html#selectNext()

selectNext can throw the following Exception:
IOException - if there is a communication problem to the reader or the Secure 
Element.
IllegalStateException - if the channel is used after being closed or it is not 
connected.
SecurityException - if the command is filtered by the security policy

While SecurityException is not document in OpenMobile 2.04, it is an OK 
behavior assuming Access Control Enforcer can be enforced correctly during the 
selectNext() call (which based on how AC enforcer is implemented currently, it 
could be an issue, but that would be another error).

However, SEEK is missing one possible Exception based on OpenMobile 2.04 for 
selectNext():
OperationNotSupportedError - if this operation is not supported

This is actually quick important, because:
1) UICC might not support openBasicChannel or openLogicalChannel against 
Partial AID, in such case, selectNext() might fail.
2) if openBasicChannel and openLogicalChannel against Partial AID is supported 
by SEEK and target UICC, it might not be possible to enforce Access Control 
correctly (as selectNext would not know about result AID in order to enforce 
the exact rule).

Hence, the SEEK documentation as well as implementation should be update to 
handle it.

Original issue reported on code.google.com by tommypo...@gmail.com on 9 Jan 2014 at 6:36

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

No branches or pull requests

1 participant