vJCRE icon indicating copy to clipboard operation
vJCRE copied to clipboard

OwnerPIN check and update issue when using an offset

Open vletoux opened this issue 8 years ago • 2 comments

in /src/javacard/framework/OwnerPIN.java (https://github.com/martinpaljak/vJCRE/blob/85c7828089fcde0fa8cde89a622b5ec582804947/src/javacard/framework/OwnerPIN.java)

The offset parameter of the functions check and update is not used. My opinion is that: Util.arrayCopy(pin, (short)0, value, (short)1, length); should be changed to Util.arrayCopy(pin, offset, value, (short)1, length);

and if (Util.arrayCompare(pin, (short) 0, value, (short) 1, length) == 0) { should be changed to if (Util.arrayCompare(pin, offset, value, (short) 1, length) == 0) {

vletoux avatar Nov 07 '15 13:11 vletoux