Fix crash on mismatching quotation (fixes #137)

docs/changes.rst

@@ -31,6 +31,9 @@ v0.7.0 (in development)
 - Fix ``gst.LinkError`` which appeared when using newer versions of GStreamer,
   e.g. on Ubuntu 12.04 Alpha. (Fixes: :issue:`144`)
 
+- Fix crash on mismatching quotation in ``list`` MPD queries. (Fixes:
+  :issue:`137`)
+
 
 v0.6.1 (2011-12-28)
 ===================

mopidy/frontends/mpd/protocol/music_db.py

@@ -189,8 +189,14 @@ def _list_build_query(field, mpd_query):
     """Converts a ``list`` query to a Mopidy query."""
     if mpd_query is None:
         return {}
-    # shlex does not seem to be friends with unicode objects
-    tokens = shlex.split(mpd_query.encode('utf-8'))
+    try:
+        # shlex does not seem to be friends with unicode objects
+        tokens = shlex.split(mpd_query.encode('utf-8'))
+    except ValueError as error:
+        if error.message == 'No closing quotation':
+            raise MpdArgError(u'Invalid unquoted character', command=u'list')
+        else:
+            raise error
     tokens = [t.decode('utf-8') for t in tokens]
     if len(tokens) == 1:
         if field == u'album':

tests/frontends/mpd/protocol/regression_test.py

@@ -146,3 +146,19 @@ class IssueGH113RegressionTest(protocol.BaseTestCase):
         self.sendRequest(
             r'listplaylistinfo "all lart spotify:track:\\w\\{22\\} pastes"')
         self.assertInResponse('OK')
+
+
+class IssueGH137RegressionTest(protocol.BaseTestCase):
+    """
+    The issue: https://github.com/mopidy/mopidy/issues/137
+
+    How to reproduce:
+
+    - Send "list" query with mismatching quotes
+    """
+
+    def test(self):
+        self.sendRequest(u'list Date Artist "Anita Ward" '
+            u'Album "This Is Remixed Hits - Mashups & Rare 12" Mixes"')
+
+        self.assertInResponse('ACK [2@0] {list} Invalid unquoted character')