refactor delete action to use ykpiv_save_object()

lib/ykpiv.c

@@ -682,6 +682,7 @@ ykpiv_rc ykpiv_save_object(ykpiv_state *state, int object_id,
   unsigned char templ[] = {0, YKPIV_INS_PUT_DATA, 0x3f, 0xff};
   int sw;
   ykpiv_rc res;
+  unsigned long outlen = 0;
 
   if(len > sizeof(data) - 9) {
     return YKPIV_SIZE_ERROR;
@@ -696,7 +697,7 @@ ykpiv_rc ykpiv_save_object(ykpiv_state *state, int object_id,
   memcpy(dataptr, indata, len);
   dataptr += len;
 
-  if((res = ykpiv_transfer_data(state, templ, data, dataptr - data, NULL, 0,
+  if((res = ykpiv_transfer_data(state, templ, data, dataptr - data, NULL, &outlen,
 	  &sw)) != YKPIV_OK) {
     return res;
   }

tool/yubico-piv-tool.c

@@ -807,32 +807,15 @@ static bool change_pin(ykpiv_state *state, enum enum_action action, const char *
 }
 
 static bool delete_certificate(ykpiv_state *state, enum enum_slot slot) {
-  unsigned char objdata[7];
-  unsigned char *ptr = objdata;
-  unsigned char data[0xff];
-  unsigned long recv_len = sizeof(data);
-  unsigned char templ[] = {0, YKPIV_INS_PUT_DATA, 0x3f, 0xff};
-  int sw;
-  bool ret = false;
   int object = get_object_id(slot);
 
-  *ptr++ = 0x5c;
+  if(ykpiv_save_object(state, object, NULL, 0) != YKPIV_OK) {
-  *ptr++ = 0x03;
+    fprintf(stderr, "Failed deleting object.\n");
-  *ptr++ = (object >> 16) & 0xff;
-  *ptr++ = (object >> 8) & 0xff;
-  *ptr++ = object & 0xff;
-  *ptr++ = 0x53;
-  *ptr++ = 0x00; /* length 0 means we'll delete the object */
-
-  if(ykpiv_transfer_data(state, templ, objdata, 7, data, &recv_len, &sw)
-      != YKPIV_OK) {
     return false;
-  } else if(sw != 0x9000) {
-    fprintf(stderr, "Failed deleting certificate to device with code %x.\n", sw);
   } else {
-    ret = true;
+    fprintf(stdout, "Certificate deleted.\n");
+    return true;
   }
-  return ret;
 }
 
 int main(int argc, char *argv[]) {