Use new plist_copy instead of serialization workaround in lockdownd_et_value()

1 files changed, 1 insertions, 11 deletions

diff --git a/src/lockdown.c b/src/lockdown.c
index 7e1f1a8..24283cb 100644
--- a/src/lockdown.c
+++ b/src/lockdown.c
@@ -445,14 +445,8 @@ iphone_error_t lockdownd_get_value(lockdownd_client_t client, const char *domain
                 plist_get_key_val(value_key_node, &result_key);
 
                 if (!strcmp(result_key, "Value")) {
-                        char *buf = NULL;
-                        uint32_t length;
                         log_dbg_msg(DBGMASK_LOCKDOWND, "lockdownd_get_value(): has a value\n");
-
-                        /* FIXME: libplist does not offer cloning of nodes, use serialization */
-                        plist_to_bin(value_value_node, &buf, &length);
-                        plist_from_bin(buf, length, value_node);
-                        free(buf);
+                        *value_node = plist_copy(value_value_node);
                 }
                 free(result_key);
         }
@@ -469,7 +463,6 @@ iphone_error_t lockdownd_get_device_uid(lockdownd_client_t client, char **uid)
 {
         iphone_error_t ret = IPHONE_E_UNKNOWN_ERROR;
         plist_t value = NULL;
-        value = plist_new_dict();
 
         ret = lockdownd_get_value(client, NULL, "UniqueDeviceID", &value);
         if (ret != IPHONE_E_SUCCESS) {
@@ -492,7 +485,6 @@ iphone_error_t lockdownd_get_device_public_key(lockdownd_client_t client, gnutls
         plist_t value = NULL;
         char *value_value = NULL;
         uint64_t size = 0;
-        value = plist_new_dict();
 
         ret = lockdownd_get_value(client, NULL, "DevicePublicKey", &value);
         if (ret != IPHONE_E_SUCCESS) {
@@ -519,14 +511,12 @@ iphone_error_t lockdownd_get_device_name(lockdownd_client_t client, char **devic
 {
         iphone_error_t ret = IPHONE_E_UNKNOWN_ERROR;
         plist_t value = NULL;
-        value = plist_new_dict();
 
         ret = lockdownd_get_value(client, NULL, "DeviceName", &value);
         if (ret != IPHONE_E_SUCCESS) {
                 return ret;
         }
         plist_get_string_val(value, device_name);
-        log_debug_msg("%s: %s\n", __func__, device_name);
 
         plist_free(value);
         value = NULL;