Attention is currently required from: dexter.

laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/aram-applet/+/39615?usp=email )

Change subject: AccessRuleMaster: allow locking of STORE DATA commands ......................................................................

Patch Set 2:

(3 comments)

File aram/src/main/java/fr/bmartel/aram/AccessRuleMaster.java:

https://gerrit.osmocom.org/c/aram-applet/+/39615/comment/5b4aafbc_e2980fbd?u... : PS2, Line 106: ISOException.throwIt(ISO7816.SW_SECURITY_STATUS_NOT_SATISFIED); why is this exception not raised in the INSTALL FOR PERSO case? The only condition for raising it is aram_lock_status, AFAICT.

https://gerrit.osmocom.org/c/aram-applet/+/39615/comment/b1725e2d_8594b6a6?u... : PS2, Line 304: this.aram_lock_status = true; mixing tab (new) and space (old) indentation

https://gerrit.osmocom.org/c/aram-applet/+/39615/comment/805d2682_1cf9cd99?u... : PS2, Line 307: this.aram_lock_status = false; where is the check that this command is only issued via the INSTALL FOR PERSO ?