forked-daapd fix to support non-apple airtunes devices

gistfile1.txt

From c8a7649c9d087132b76fe9b296daaa00c85e0ce6 Mon Sep 17 00:00:00 2001
From: Stuart C Naifeh <scnaifeh@ruby.naifeh.org>
Date: Mon, 18 Nov 2013 12:46:45 -0500
Subject: [PATCH 3/3] fix to support non-apple airtunes devices

---
 src/player.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
 src/raop.c   | 12 ++++--------
 src/raop.h   |  2 ++
 3 files changed, 50 insertions(+), 8 deletions(-)

diff --git a/src/player.c b/src/player.c
index 1242420..601cbea 100644
--- a/src/player.c
+++ b/src/player.c
@@ -3532,6 +3532,8 @@ raop_device_cb(const char *name, const char *type, const char *domain, const cha
   char *at_name;
   char *password;
   uint64_t id;
+  char encrypt;
+  char wants_metadata;
   char has_password;
   enum raop_devtype devtype;
   int ret;
@@ -3662,6 +3664,46 @@ raop_device_cb(const char *name, const char *type, const char *domain, const cha
     devtype = RAOP_DEV_APPLETV;
 
  no_am:
+  p = keyval_get(txt, "ek");
+  if (!p)
+    {
+      DPRINTF(E_LOG, L_PLAYER, "AirTunes %s: no ek field in TXT record!\n", name);
+      encrypt = 0;
+
+      goto no_ek;
+    }
+
+  if (*p == '\0')
+    {
+      DPRINTF(E_LOG, L_PLAYER, "AirTunes %s: ek has no value\n", name);
+      encrypt = 0;
+
+      goto no_ek;
+    }
+
+  encrypt = (*p == '1') ? 1: 0;
+
+ no_ek:
+  p = keyval_get(txt, "md");
+  if (!p)
+    {
+      DPRINTF(E_LOG, L_PLAYER, "AirTunes %s: no md field in TXT record!\n", name);
+      wants_metadata = 0;
+
+      goto no_md;
+    }
+
+  if (*p == '\0')
+    {
+      DPRINTF(E_LOG, L_PLAYER, "AirTunes %s: md has no value\n", name);
+      wants_metadata = 0;
+
+      goto no_md;
+    }
+
+    wants_metadata = 1;
+
+ no_md:
   DPRINTF(E_DBG, L_PLAYER, "AirTunes device %s: password: %s, type %s\n", name, (password) ? "yes" : "no", raop_devtype[devtype]);
 
   rd->advertised = 1;
@@ -3681,6 +3723,8 @@ raop_device_cb(const char *name, const char *type, const char *domain, const cha
 
   rd->devtype = devtype;
 
+  rd->encrypt = encrypt;
+  rd->wants_metadata = wants_metadata;
   rd->has_password = has_password;
   rd->password = password;
 
diff --git a/src/raop.c b/src/raop.c
index 168ceed..8a01c9e 100644
--- a/src/raop.c
+++ b/src/raop.c
@@ -1778,26 +1778,22 @@ raop_session_make(struct raop_device *rd, int family, raop_status_cb cb)
   rs->status_cb = cb;
   rs->server_fd = -1;
 
+  rs->encrypt = rd->encrypt;
+  rs->wants_metadata = rd->wants_metadata;
   rs->password = rd->password;
 
   switch (rd->devtype)
     {
       case RAOP_DEV_APEX_80211G:
-	rs->encrypt = 1;
-	rs->auth_quirk_itunes = 1;
-	rs->wants_metadata = 0;
-	break;
+        rs->auth_quirk_itunes = 1;
+        break;
 
       case RAOP_DEV_APEX_80211N:
-	rs->encrypt = 1;
 	rs->auth_quirk_itunes = 0;
-	rs->wants_metadata = 0;
 	break;
 
       case RAOP_DEV_APPLETV:
-	rs->encrypt = 0;
 	rs->auth_quirk_itunes = 0;
-	rs->wants_metadata = 1;
 	break;
     }
 
diff --git a/src/raop.h b/src/raop.h
index cb1000d..bdebad4 100644
--- a/src/raop.h
+++ b/src/raop.h
@@ -26,6 +26,8 @@ struct raop_device
   unsigned selected:1;
   unsigned advertised:1;
 
+  unsigned encrypt:1;
+  unsigned wants_metadata:1;
   unsigned has_password:1;
   const char *password;
 
-- 
1.8.3.2