Fixed star parser, prevent him from going into an infinite loop if an early line starts with a "d"

modules/io/src/mol/star_parser.cc

@@ -550,6 +550,8 @@ void StarParser::Parse()
       case 'd':
         if (tline.length()>=5 && StringRef("data_", 5)==tline.substr(0, 5)) {
           this->ParseData();
+        } else {
+          throw IOException("Missing 'data_' control structure");
         }
         break;
       case 'g':

modules/io/tests/test_star_parser.cc

@@ -389,6 +389,13 @@ BOOST_AUTO_TEST_CASE(star_items_as_row)
   BOOST_TEST_MESSAGE("  done.");
 }
 
+BOOST_AUTO_TEST_CASE(star_broken_data)
+{
+  std::ifstream s("testfiles/broken_data.cif");
+  LoopTestParser star_p(s);
+  BOOST_CHECK_THROW(star_p.Parse(), IOException);
+}
+
 BOOST_AUTO_TEST_CASE(star_missing_data)
 {
   std::ifstream s("testfiles/missing_data.cif");

modules/io/tests/testfiles/broken_data.cif

+d lines starting with a 'd' kicked the parser into an infinite loop.