Changeset 10350

Timestamp:

04/10/11 10:39:54 (14 years ago)

Author:

tbretz

Message:

Throw logic_error instead of runtime_error and throw in case the description contains a EOL character.

File:

• trunk/FACT++/src/DimDescriptionService.cc (modified) (2 diffs)

trunk/FACT++/src/DimDescriptionService.cc

@@ -62 +62 @@
 //!
 //! @throws
-//!     If a server name couldn't be reliably determined a runtime_error
-//!     exception is thrown.
+//!     If a server name couldn't be reliably determined a logic_error
+//!     exception is thrown; if the given description contains a '\n'
+//!     also a logic_error is thrown.
 //
 DimDescriptionService::DimDescriptionService(const std::string &name, const std::string &desc)
@@ -72 +73 @@
         const size_t p = name.find_first_of('/');
         if (p==string::npos)
-            throw runtime_error("Could not determine server name");
+            throw logic_error("Could not determine server name");
 
         service = name.substr(0, p);
     }
+
+    if (desc.find_first_of('\n')!=string::npos)
+            throw logic_error("Description for "+name+" contains '\\n'");
 
     service += "/SERVICE_DESC";