From 3537f78a98111b88aab5e2d94dda340cf09be5af Mon Sep 17 00:00:00 2001
From: Nikias Bassen
Date: Fri, 13 Dec 2013 04:14:53 +0100
Subject: make libplist/protov1 support mandatory

---
 src/client.c | 32 +++-----------------------------
 1 file changed, 3 insertions(+), 29 deletions(-)

(limited to 'src')

diff --git a/src/client.c b/src/client.c
index ac5d08e..b81c11c 100644
--- a/src/client.c
+++ b/src/client.c
@@ -33,9 +33,7 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
 #include <arpa/inet.h>
 #include <pthread.h>
 
-#ifdef HAVE_PLIST
 #include <plist/plist.h>
-#endif
 
 #include "log.h"
 #include "usb.h"
@@ -186,7 +184,6 @@ static int send_pkt(struct mux_client *client, uint32_t tag, enum usbmuxd_msgtyp
 	return hdr.length;
 }
 
-#ifdef HAVE_PLIST
 static int send_plist_pkt(struct mux_client *client, uint32_t tag, plist_t plist)
 {
 	int res = -1;
@@ -201,12 +198,10 @@ static int send_plist_pkt(struct mux_client *client, uint32_t tag, plist_t plist
 	}
 	return res;
 }
-#endif
 
 static int send_result(struct mux_client *client, uint32_t tag, uint32_t result)
 {
 	int res = -1;
-#ifdef HAVE_PLIST
 	if (client->proto_version == 1) {
 		/* XML plist packet */
 		plist_t dict = plist_new_dict();
@@ -214,9 +209,7 @@ static int send_result(struct mux_client *client, uint32_t tag, uint32_t result)
 		plist_dict_insert_item(dict, "Number", plist_new_uint(result));
 		res = send_plist_pkt(client, tag, dict);
 		free(dict);
-	} else
-#endif
-	{
+	} else {
 		/* binary packet */
 		res = send_pkt(client, tag, MESSAGE_RESULT, &result, sizeof(uint32_t));
 	}
@@ -246,7 +239,6 @@ int client_notify_connect(struct mux_client *client, enum usbmuxd_result result)
 	return 0;
 }
 
-#ifdef HAVE_PLIST
 static plist_t create_device_attached_plist(struct device_info *dev)
 {
 	plist_t dict = plist_new_dict();
@@ -292,20 +284,16 @@ static int send_device_list(struct mux_client *client, uint32_t tag)
 	plist_free(dict);
 	return res;
 }
-#endif
 
 static int notify_device_add(struct mux_client *client, struct device_info *dev)
 {
 	int res = -1;
-#ifdef HAVE_PLIST
 	if (client->proto_version == 1) {
 		/* XML plist packet */
 		plist_t dict = create_device_attached_plist(dev);
 		res = send_plist_pkt(client, 0, dict);
 		plist_free(dict);
-	} else
-#endif
-	{
+	} else {
 		/* binary packet */
 		struct usbmuxd_device_record dmsg;
 		memset(&dmsg, 0, sizeof(dmsg));
@@ -322,7 +310,6 @@ static int notify_device_add(struct mux_client *client, struct device_info *dev)
 static int notify_device_remove(struct mux_client *client, uint32_t device_id)
 {
 	int res = -1;
-#ifdef HAVE_PLIST
 	if (client->proto_version == 1) {
 		/* XML plist packet */
 		plist_t dict = plist_new_dict();
@@ -330,9 +317,7 @@ static int notify_device_remove(struct mux_client *client, uint32_t device_id)
 		plist_dict_insert_item(dict, "DeviceID", plist_new_uint(device_id));
 		res = send_plist_pkt(client, 0, dict);
 		plist_free(dict);
-	} else
-#endif
-	{
+	} else {
 		/* binary packet */
 		res = send_pkt(client, 0, MESSAGE_DEVICE_REMOVE, &device_id, sizeof(uint32_t));
 	}
@@ -383,25 +368,17 @@ static int client_command(struct mux_client *client, struct usbmuxd_header *hdr)
 		return -1;
 	}
 
-#ifdef HAVE_PLIST
 	if((hdr->version != 0) && (hdr->version != 1)) {
 		usbmuxd_log(LL_INFO, "Client %d version mismatch: expected 0 or 1, got %d", client->fd, hdr->version);
-#else
-	if(hdr->version != USBMUXD_PROTOCOL_VERSION) {
-		usbmuxd_log(LL_INFO, "Client %d version mismatch: expected %d, got %d", client->fd, USBMUXD_PROTOCOL_VERSION, hdr->version);
-#endif
 		send_result(client, hdr->tag, RESULT_BADVERSION);
 		return 0;
 	}
 
 	struct usbmuxd_connect_request *ch;
-#ifdef HAVE_PLIST
 	char *payload;
 	uint32_t payload_size;
-#endif
 
 	switch(hdr->message) {
-#ifdef HAVE_PLIST
 		case MESSAGE_PLIST:
 			client->proto_version = 1;
 			payload = (char*)(hdr) + sizeof(struct usbmuxd_header);
@@ -469,12 +446,10 @@ static int client_command(struct mux_client *client, struct usbmuxd_header *hdr)
 						client->state = CLIENT_CONNECTING1;
 					}
 					return 0;
-#ifdef HAVE_PLIST
 				} else if (!strcmp(message, "ListDevices")) {
 					if (send_device_list(client, hdr->tag) < 0)
 						return -1;
 					return 0;
-#endif
 				} else {
 					usbmuxd_log(LL_ERROR, "Unexpected command '%s' received!", message);
 					free(message);
@@ -486,7 +461,6 @@ static int client_command(struct mux_client *client, struct usbmuxd_header *hdr)
 			}
 			// should not be reached?!
 			return -1;
-#endif
 		case MESSAGE_LISTEN:
 			if(send_result(client, hdr->tag, 0) < 0)
 				return -1;
-- 
cgit v1.1-32-gdbae