Bluez5: Fix plain text handover of xml from sdpscanner

The plain text option was never triggered as the raw data processing
always considered \0 as part of the string. Subsequently the string
was always evaluated to be non-printable and the hex based handover
was triggered. Fortunately the hex based handover worked without
trouble.

In addition the plain value was not properly escaped to be valid XML.

Change-Id: I537a94ef4705c7eeab143b3053affc6abbd548cc
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>

1 files changed, 10 insertions, 6 deletions

diff --git a/src/tools/sdpscanner/main.cpp b/src/tools/sdpscanner/main.cpp
index 5cf4ceb0..cb1bdd15 100644
--- a/src/tools/sdpscanner/main.cpp
+++ b/src/tools/sdpscanner/main.cpp
@@ -141,9 +141,13 @@ static void parseAttributeValues(sdp_data_t *data, int indentation, QByteArray &
         QByteArray text = QByteArray::fromRawData(data->val.str, data->unitSize);
 
         bool hasNonPrintableChar = false;
-        for (int i = 0; i < text.count() && !hasNonPrintableChar; i++) {
-            if (!isprint(text[i])) {
+        for (int i = 0; i < text.count(); i++) {
+            if (text[i] == '\0') {
+                text.resize(i); // cut trailing content
+                break;
+            } else if (!isprint(text[i])) {
                 hasNonPrintableChar = true;
+                text.resize(text.indexOf('\0')); // cut trailing content
                 break;
             }
         }
@@ -152,10 +156,10 @@ static void parseAttributeValues(sdp_data_t *data, int indentation, QByteArray &
             xmlOutput.append("encoding=\"hex\" value=\"");
             xmlOutput.append(text.toHex());
         } else {
-            text.replace('&', "&amp");
-            text.replace('<', "&lt");
-            text.replace('>', "&gt");
-            text.replace('"', "&quot");
+            text.replace('&', "&amp;");
+            text.replace('<', "&lt;");
+            text.replace('>', "&gt;");
+            text.replace('"', "&quot;");
 
             xmlOutput.append("value=\"");
             xmlOutput.append(text);