Use defined return value

This commit is contained in:
wh201906 2023-11-25 11:09:16 +08:00
commit 10ab013f5b
No known key found for this signature in database
2 changed files with 4 additions and 4 deletions

View file

@ -1402,7 +1402,7 @@ int ExchangeAPDUSC(bool verbose, uint8_t *datain, int datainlen, bool activateCa
int len = smart_responseEx(dataout, maxdataoutlen, verbose); int len = smart_responseEx(dataout, maxdataoutlen, verbose);
if (len < 0) { if (len < 0) {
free(payload); free(payload);
return 1; return PM3_ESOFT;
} }
// retry // retry
@ -1419,13 +1419,13 @@ int ExchangeAPDUSC(bool verbose, uint8_t *datain, int datainlen, bool activateCa
len = smart_responseEx(dataout, maxdataoutlen, verbose); len = smart_responseEx(dataout, maxdataoutlen, verbose);
if (len < 0) { if (len < 0) {
free(payload); free(payload);
return 1; return PM3_ESOFT;
} }
} }
free(payload); free(payload);
*dataoutlen = len; *dataoutlen = len;
return 0; return PM3_SUCCESS;
} }
bool smart_select(bool verbose, smart_card_atr_t *atr) { bool smart_select(bool verbose, smart_card_atr_t *atr) {

View file

@ -153,7 +153,7 @@ int Iso7816ExchangeEx(Iso7816CommandChannel channel, bool activate_field, bool l
res = ExchangeAPDUSC(false, data, datalen, activate_field, leave_field_on, result, (int)max_result_len, (int *)result_len); res = ExchangeAPDUSC(false, data, datalen, activate_field, leave_field_on, result, (int)max_result_len, (int *)result_len);
} }
if (res) { if (res != PM3_SUCCESS) {
return res; return res;
} }
break; break;