From c57b95b53ed6508516007e0fc7f7b4058f7a6748 Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Fri, 15 Mar 2019 16:27:50 +0000
Subject: [PATCH] Give better errors on malformed subrip times.

---
 src/subrip_reader.cc | 33 +++++++++++++++++++++++++++------
 1 file changed, 27 insertions(+), 6 deletions(-)

diff --git a/src/subrip_reader.cc b/src/subrip_reader.cc
index 5625679..b547db0 100644
--- a/src/subrip_reader.cc
+++ b/src/subrip_reader.cc
@@ -160,12 +160,33 @@ SubripReader::convert_time (string t)
 		throw SubripError (t, "time in the format h:m:s,ms", _context);
 	}
 
-	return Time::from_hms (
-		lexical_cast<int> (a[0]),
-		lexical_cast<int> (a[1]),
-		lexical_cast<int> (b[0]),
-		lexical_cast<int> (b[1])
-		);
+	int h, m, s, ms;
+
+	try {
+		h = lexical_cast<int>(a[0]);
+	} catch (boost::bad_lexical_cast &) {
+		throw SubripError (t, "integer hour value", _context);
+	}
+
+	try {
+		m = lexical_cast<int>(a[1]);
+	} catch (boost::bad_lexical_cast &) {
+		throw SubripError (t, "integer minute value", _context);
+	}
+
+	try {
+		s = lexical_cast<int>(b[0]);
+	} catch (boost::bad_lexical_cast &) {
+		throw SubripError (t, "integer second value", _context);
+	}
+
+	try {
+		ms = lexical_cast<int>(b[1]);
+	} catch (boost::bad_lexical_cast &) {
+		throw SubripError (t, "integer millisecond value", _context);
+	}
+
+	return Time::from_hms (h, m, s, ms);
 }
 
 void
-- 
2.30.2