leoslyrics plugin: handle parser exceptions

Should fix bug #2286. Added exception handling for both calls of the SAX
XML parser. Also removed trailing newlines from the result to keep us from
scrolling down further than the last line.

diff --git a/lyrics/03-leoslyrics.py b/lyrics/03-leoslyrics.py
index b55f0ec67087d2a0d06131a7943fc31148620c34..2a40f37a8bab6833a60d248652efa4d5eabf0a1f 100755 (executable)
--- a/lyrics/03-leoslyrics.py
+++ b/lyrics/03-leoslyrics.py
@@ -21,7 +21,7 @@
 # Load lyrics from leoslyrics.com
 #
 
-from sys import argv, exit
+from sys import argv, exit, stderr
 from urllib import urlencode, urlopen
 from xml.sax import make_parser, SAXException
 from xml.sax.handler import ContentHandler
@@ -47,7 +47,11 @@ def search(artist, title):
     handler = SearchContentHandler()
     parser = make_parser()
     parser.setContentHandler(handler)
-    parser.parse(f)
+    try:
+        parser.parse(f)
+    except SAXException:
+        stderr.write("Failed to parse the search result!\n")
+        exit(1)
     return handler.hid
 
 class LyricsContentHandler(ContentHandler):
@@ -75,10 +79,14 @@ def lyrics(hid):
     handler = LyricsContentHandler()
     parser = make_parser()
     parser.setContentHandler(handler)
-    parser.parse(f)
+    try:
+        parser.parse(f)
+    except SAXException:
+        stderr.write("Failed to parse the lyrics!\n")
+        exit(1)
     return handler.text
 
 hid = search(argv[1], argv[2])
 if hid is None:
     exit(69)
-print lyrics(hid).encode('utf-8')
+print lyrics(hid).encode('utf-8').rstrip()