james/yubikey.rs
Public Access
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix broken unblock-pin action

Browse Source 
the unblock pin action misstakenly used pin reference 0x81 (unblock)
instead of 0x80 (pin)
This commit is contained in:
Klas Lindfors
2014-10-29 08:09:17 +01:00
parent 146fa881f2
commit ccf9d01027
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tool/yubico-piv-tool.c
+3 -3
View File
@@ -883,7 +883,7 @@ static bool verify_pin(ykpiv_state *state, const char *pin) {
* since they're very similar in what data they use. */ * since they're very similar in what data they use. */
static bool change_pin(ykpiv_state *state, enum enum_action action, const char *pin, static bool change_pin(ykpiv_state *state, enum enum_action action, const char *pin,
const char *new_pin) { const char *new_pin) {
unsigned char templ[] = {0, YKPIV_INS_CHANGE_REFERENCE, 0, 0x81}; unsigned char templ[] = {0, YKPIV_INS_CHANGE_REFERENCE, 0, 0x80};
unsigned char indata[0x10]; unsigned char indata[0x10];
unsigned char data[0xff]; unsigned char data[0xff];
unsigned long recv_len = sizeof(data); unsigned long recv_len = sizeof(data);
@@ -899,8 +899,8 @@ static bool change_pin(ykpiv_state *state, enum enum_action action, const char *
if(action == action_arg_unblockMINUS_pin) { if(action == action_arg_unblockMINUS_pin) {
templ[1] = YKPIV_INS_RESET_RETRY; templ[1] = YKPIV_INS_RESET_RETRY;
} }
else if(action == action_arg_changeMINUS_pin) { else if(action == action_arg_changeMINUS_puk) {
templ[3] = 0x80; templ[3] = 0x81;
} }
memcpy(indata, pin, pin_len); memcpy(indata, pin, pin_len);
if(pin_len < 8) { if(pin_len < 8) {