[sc-lib] Add channel number checks

Author: kostorr

src/Ic.cxx

@@ -74,7 +74,7 @@ Ic::Ic(std::string cardId, int linkId)
 
 void Ic::init(const roc::Parameters::CardIdType& cardId, int linkId)
 {
-  if (mLink.linkId >= CRU_NUM_LINKS) {
+  if (linkId >= CRU_NUM_LINKS) {
     BOOST_THROW_EXCEPTION(
       IcException() << ErrorInfo::Message("Maximum link number exceeded"));
   }
@@ -99,6 +99,11 @@ void Ic::init(const roc::Parameters::CardIdType& cardId, int linkId)
 
 void Ic::setChannel(int gbtChannel)
 {
+  if (gbtChannel >= CRU_NUM_LINKS) {
+    BOOST_THROW_EXCEPTION(
+      IcException() << ErrorInfo::Message("Maximum link number exceeded"));
+  }
+
   mLink.linkId = gbtChannel;
   mLink.rawLinkId = mLink.serialId.getEndpoint() * 12 + gbtChannel;
   barWrite(sc_regs::SC_LINK.index, mLink.rawLinkId);

src/Sca.cxx

@@ -78,6 +78,11 @@ void Sca::init(const roc::Parameters::CardIdType& cardId, int linkId)
 
 void Sca::setChannel(int gbtChannel)
 {
+  if (gbtChannel >= CRU_NUM_LINKS) {
+    BOOST_THROW_EXCEPTION(
+      ScaException() << ErrorInfo::Message("Maximum link number exceeded"));
+  }
+
   mLink.linkId = gbtChannel;
   mLink.rawLinkId = mLink.serialId.getEndpoint() * 12 + gbtChannel;
   barWrite(sc_regs::SC_LINK.index, mLink.rawLinkId);

src/Swt.cxx

@@ -59,7 +59,7 @@ Swt::Swt(std::string cardId, int linkId)
 
 void Swt::init(const roc::Parameters::CardIdType& cardId, int linkId)
 {
-  if (mLink.linkId >= CRU_NUM_LINKS) {
+  if (linkId >= CRU_NUM_LINKS) {
     BOOST_THROW_EXCEPTION(
       SwtException() << ErrorInfo::Message("Maximum link number exceeded"));
   }
@@ -81,6 +81,11 @@ void Swt::init(const roc::Parameters::CardIdType& cardId, int linkId)
 
 void Swt::setChannel(int gbtChannel)
 {
+  if (gbtChannel >= CRU_NUM_LINKS) {
+    BOOST_THROW_EXCEPTION(
+      SwtException() << ErrorInfo::Message("Maximum link number exceeded"));
+  }
+
   mLink.linkId = gbtChannel;
   mLink.rawLinkId = mLink.serialId.getEndpoint() * 12 + gbtChannel;
   barWrite(sc_regs::SC_LINK.index, mLink.rawLinkId);