Programming answers in double ended queue. Fix word "Coding" by "Encoding"
[anna.git] / source / diameter / codec / Message.cpp
index 7fc92e8..b286fbc 100644 (file)
@@ -59,6 +59,7 @@
 
 // STL
 #include <string>
+#include <vector>
 
 
 using namespace anna;
@@ -226,6 +227,20 @@ void Message::setId(const char *name) throw(anna::RuntimeException) {
 }
 
 
+//------------------------------------------------------------------------------
+//-------------------------------------------------- Message::setApplicationId()
+//------------------------------------------------------------------------------
+void Message::setApplicationId(U32 aid) throw() {
+  a_applicationId = aid;
+
+  // Default behaviour:
+  if (!getEngine()->hasSelectStackWithApplicationId()) return;
+
+  // Adapts for Application-ID stack identifier:
+  getEngine()->setDictionary(aid);
+}
+
+
 //------------------------------------------------------------------------------
 //------------------------------------------------------------ Message::addAvp()
 //------------------------------------------------------------------------------
@@ -293,7 +308,7 @@ void Message::decode(const anna::DataBlock &db, Message *ptrAnswer) throw(anna::
   //
   // VALIDATION PHASE
   // Launch exception on first validation error (validateAll == false), or log warning reporting all validation errors when
-  //  complete validation is desired (validateAll == true, engine default) launching a final exception like "decoded an invalid message".
+  //  complete validation is desired (validateAll == true, engine default) launching a final exception like "the decoded message is invalid".
   // OAM
   OamModule &oamModule = OamModule::instantiate();
 
@@ -330,9 +345,9 @@ void Message::decode(const anna::DataBlock &db, Message *ptrAnswer) throw(anna::
 
   U24 code = DECODE3BYTES_INDX_VALUETYPE(buffer, 5, U24);
 
-  setId(CommandId(code, requestBit() /* based on a_flags */));
+  a_id = CommandId(code, requestBit() /* based on a_flags */);
 
-  a_applicationId = DECODE4BYTES_INDX_VALUETYPE(buffer, 8, U32);
+  setApplicationId(DECODE4BYTES_INDX_VALUETYPE(buffer, 8, U32)); // centralize set, because it could be used for stack selection.
 
   a_hopByHop = DECODE4BYTES_INDX_VALUETYPE(buffer, 12, U32);
 
@@ -369,16 +384,17 @@ void Message::decode(const anna::DataBlock &db, Message *ptrAnswer) throw(anna::
   Avp* avp;
   anna::DataBlock db_aux;
 
+  // Parent information:
+  parent_t parent;
+  parent.setMessage(a_id);
+
+
   while(avpPos < dataBytes) {
     try {
       avp =  getEngine()->allocateAvp();
       db_aux.assign(startData + avpPos, dataBytes - avpPos /* is valid to pass total length (indeed i don't know the real avp length) because it will be limited and this has deep copy disabled (no memory is reserved) */);
-      avp -> decode(db_aux, answer);
+      avp -> decode(db_aux, parent, answer);
     } catch(anna::RuntimeException &ex) {
-      if(answer) {
-        //answer->setResultCode(helpers::base::AVPVALUES__Result_Code::DIAMETER_INVALID_AVP_VALUE); // unspecified error ...
-        answer->setFailedAvp(avp->getId());
-      }
       getEngine()->releaseAvp(avp);
       LOGWARNING(
         anna::Logger::warning(ex.getText(), ANNA_FILE_LOCATION);
@@ -407,7 +423,7 @@ void Message::decode(const anna::DataBlock &db, Message *ptrAnswer) throw(anna::
 
   if((vmode == Engine::ValidationMode::AfterDecoding) || (vmode == Engine::ValidationMode::Always))
     if(!valid(answer))
-      throw anna::RuntimeException("Decoded an invalid message. See previous report on warning-level traces", ANNA_FILE_LOCATION);
+      throw anna::RuntimeException("The decoded message is invalid. See previous report on warning-level traces", ANNA_FILE_LOCATION);
 }
 
 
@@ -455,25 +471,12 @@ int Message::getResultCode() const throw() {
 }
 
 
-//------------------------------------------------------------------------------
-//--------------------------------------------------- Message::addTheFailedAVP()
-//------------------------------------------------------------------------------
-Avp * Message::addTheFailedAVP() throw() {
-  Avp *result = getAvp(helpers::base::AVPID__Failed_AVP, 1, anna::Exception::Mode::Ignore);
-
-  // Section 7.5 RFC 6733: A Diameter message SHOULD contain one Failed-AVP AVP
-  if(!result) result = addAvp(helpers::base::AVPID__Failed_AVP);
-
-  return result;
-}
-
-
 //------------------------------------------------------------------------------
 //------------------------------------------------------ Message::setFailedAvp()
 //------------------------------------------------------------------------------
-Avp * Message::setFailedAvp(AvpId id) throw(anna::RuntimeException) {
+void Message::setFailedAvp(const parent_t &parent, AvpId wrong, const char *wrongName) throw(anna::RuntimeException) {
 
-  if(isRequest()) return NULL;
+  if(isRequest()) return;
 
 // RFC 6733:
 //
@@ -494,44 +497,30 @@ Avp * Message::setFailedAvp(AvpId id) throw(anna::RuntimeException) {
   // Probably the RFC wants to give freedom to the application layer, but it is recommended to
   // have only one child (wrong avp) inside a unique message Failed-AVP to ease the Result-Code
   // correspondence. Anyway, this behaviour could be easily  opened commenting condition block (*).
-  Avp *theFailedAvp = addTheFailedAVP();
-  LOGDEBUG(
-    std::string msg = "Adding wrong avp ";
-    msg += anna::diameter::functions::avpIdAsPairString(id);
-    msg += " within the message Failed-AVP ...";
-    anna::Logger::debug(msg, ANNA_FILE_LOCATION);
-  );
-
-  if (theFailedAvp->countChilds()) { // (*)
-       LOGDEBUG(anna::Logger::debug("Discarding wrong avp. A previous wrong avp was already added into the Failed-AVP. RFC 6733 Section 7.5 recommends to store only the first.", ANNA_FILE_LOCATION));
-       return NULL;
+  Avp *theFailedAvp = getAvp(helpers::base::AVPID__Failed_AVP, 1, anna::Exception::Mode::Ignore);
+  if (theFailedAvp) {
+       LOGDEBUG(anna::Logger::debug("Failed-AVP has already been added. RFC 6733 Section 7.5 recommends to store only the first error found", ANNA_FILE_LOCATION));
+    return;
   }
 
-  return (theFailedAvp->addAvp(id));
-}
-
-
-//------------------------------------------------------------------------------
-//------------------------------------------------------ Message::setFailedAvp()
-//------------------------------------------------------------------------------
-Avp * Message::setFailedAvp(Avp *avp) throw(anna::RuntimeException) {
-
-  if(!avp || isRequest()) return NULL;
+  // Section 7.5 RFC 6733: A Diameter message SHOULD contain one Failed-AVP AVP
+  theFailedAvp = addAvp(helpers::base::AVPID__Failed_AVP);
+  Avp *leaf = theFailedAvp;
 
-  Avp *theFailedAvp = addTheFailedAVP();
   LOGDEBUG(
-    std::string msg = "Adding wrong avp ";
-    msg += anna::diameter::functions::avpIdAsPairString(avp->getId());
-    msg += " within the message Failed-AVP ...";
+    std::string msg = "Adding to Failed-AVP, the wrong avp ";
+    msg += wrongName ? wrongName : (anna::diameter::functions::avpIdAsPairString(wrong));
+    msg += " found inside ";
+    msg += parent.asString();
+
     anna::Logger::debug(msg, ANNA_FILE_LOCATION);
   );
 
-  if (theFailedAvp->countChilds()) { // (*)
-       LOGDEBUG(anna::Logger::debug("Discarding wrong avp. A previous wrong avp was already added into the Failed-AVP. RFC 6733 Section 7.5 recommends to store only the first.", ANNA_FILE_LOCATION));
-       return NULL;
-  }
+  std::vector<AvpId>::const_iterator it;
+  for(it = parent.AvpsId.begin(); it != parent.AvpsId.end(); it++)
+       leaf = leaf->addAvp(*it);
 
-  return (theFailedAvp->addAvp(avp));
+  leaf->addAvp(wrong);
 }
 
 
@@ -611,7 +600,6 @@ bool Message::valid(Message *ptrAnswer) const throw(anna::RuntimeException) {
   OamModule &oamModule = OamModule::instantiate();
   // Dictionary stack command:
   const stack::Command *stackCommand = getStackCommand();
-  std::string me;
   // Only build answer for a request:
   Message *answer = isRequest() ? ptrAnswer : NULL;
 
@@ -619,7 +607,7 @@ bool Message::valid(Message *ptrAnswer) const throw(anna::RuntimeException) {
 
   if(!stackCommand) {
     // OAM
-    me = anna::diameter::functions::commandIdAsPairString(a_id);
+    std::string me = anna::diameter::functions::commandIdAsPairString(a_id);
     oamModule.activateAlarm(OamModule::Alarm::MessageValidation__UnknownOperation__s__UnableToValidate, STRING_WITH_QUOTATION_MARKS__C_STR(me));
     oamModule.count(OamModule::Counter::MessageValidation__UnknownOperationUnableToValidate);
 
@@ -629,7 +617,10 @@ bool Message::valid(Message *ptrAnswer) const throw(anna::RuntimeException) {
     return false;
   }
 
-  me = stackCommand->getName();
+  // Parent information:
+  parent_t me;
+  me.setMessage(a_id, stackCommand->getName().c_str());
+
   //////////////////////////////
   // Flags coherence checking //
   //////////////////////////////
@@ -638,12 +629,12 @@ bool Message::valid(Message *ptrAnswer) const throw(anna::RuntimeException) {
 
   if(!result) {
     // OAM & Depth management
-    oamModule.activateAlarm(OamModule::Alarm::MessageValidation__Operation__s__HaveIncoherentFlags__d__, STRING_WITH_QUOTATION_MARKS__C_STR(me), (int)a_flags);
+    oamModule.activateAlarm(OamModule::Alarm::MessageValidation__Operation__s__HaveIncoherentFlags__d__, STRING_WITH_QUOTATION_MARKS__C_STR(me.asString()), (int)a_flags);
     oamModule.count(OamModule::Counter::MessageValidation__OperationHaveIncoherentFlags);
 
     if(answer) answer->setResultCode(rc);
 
-    getEngine()->validationAnomaly(anna::functions::asString("Operation %s have incoherent flags (%d)", STRING_WITH_QUOTATION_MARKS__C_STR(me), (int)a_flags));
+    getEngine()->validationAnomaly(anna::functions::asString("Operation %s have incoherent flags (%d)", STRING_WITH_QUOTATION_MARKS__C_STR(me.asString()), (int)a_flags));
   }
 
   ////////////////////
@@ -668,7 +659,7 @@ const anna::DataBlock & Message::code() throw(anna::RuntimeException) {
   // Pre-Validation
   Engine::ValidationMode::_v vmode = getEngine()->getValidationMode();
 
-  if((vmode == Engine::ValidationMode::BeforeCoding) || (vmode == Engine::ValidationMode::Always)) {
+  if((vmode == Engine::ValidationMode::BeforeEncoding) || (vmode == Engine::ValidationMode::Always)) {
     if(!valid())
       throw anna::RuntimeException("Try to encode an invalid message. See previous report on warning-level traces", ANNA_FILE_LOCATION);
   }
@@ -676,7 +667,7 @@ const anna::DataBlock & Message::code() throw(anna::RuntimeException) {
   // Pre-Fixing
   Engine::FixMode::_v fmode = getEngine()->getFixMode();
 
-  if((fmode == Engine::FixMode::BeforeCoding) || (fmode == Engine::FixMode::Always)) fix();
+  if((fmode == Engine::FixMode::BeforeEncoding) || (fmode == Engine::FixMode::Always)) fix();
 
   // Trace
   LOGDEBUG(
@@ -779,6 +770,9 @@ void Message::fromXML(const anna::xml::Node* messageNode) throw(anna::RuntimeExc
   int i_aux;
   unsigned int u_aux;
 
+  // Clear the message
+  clear();
+
   if(version) {
     i_aux = version->getIntegerValue();
 
@@ -791,6 +785,10 @@ void Message::fromXML(const anna::xml::Node* messageNode) throw(anna::RuntimeExc
     a_version = i_aux;
   }
 
+  // Application-id
+  // This is called before any operation which needs to know about the stack elements (this could set the dictionary)
+  setApplicationId(appid->getIntegerValue());
+
   // Dictionary
   const stack::Dictionary * dictionary = getEngine()->getDictionary();
   const stack::Command *stackCommand = NULL;
@@ -820,7 +818,7 @@ void Message::fromXML(const anna::xml::Node* messageNode) throw(anna::RuntimeExc
       throw anna::RuntimeException(msg, ANNA_FILE_LOCATION);
     }
 
-    setId(stackCommand->getId());
+    setId(stackCommand->getId(), false /* don't clear */);
     // 'P', 'E' and 'T' flags:
     bool activateP = pbit ? (pbit->getValue() == "yes") : false;
     bool activateE = ebit ? (ebit->getValue() == "yes") : false;
@@ -871,24 +869,11 @@ void Message::fromXML(const anna::xml::Node* messageNode) throw(anna::RuntimeExc
     a_flags = i_aux;
     int flagsBCK = a_flags;
     // Final assignments
-    setId(CommandId(u_code, requestBit() /* based on a_flags */));
+    a_id = CommandId(u_code, requestBit() /* based on a_flags */);
     // Flags could have been updated regarding dictionary, but during parsing we must respect xml file:
     a_flags = flagsBCK;
   }
 
-  // Application-id
-  u_aux = appid->getIntegerValue();
-
-  /*
-  if(u_aux < 0) {
-    std::string msg = "Error processing command <application-id '"; msg += appid->getValue();
-    msg += "': negative values are not allowed";
-    throw anna::RuntimeException(msg, ANNA_FILE_LOCATION);
-  }
-  */
-
-  setApplicationId(u_aux);
-
   // Hob-by-hop-id
   if(hbh) {
     u_aux = hbh->getIntegerValue();