AT90CAN128 - nicht definiertes Verhalten

[Jan H. (janiiix3)]

Nabend,

Tausche aktuell CAN Messages zwischen dem Atmel und dem Rechner aus.
Soweit so gut. Ab und zu kommen Messages die eigentlich keiner sendet.

inline void _CanCopyMessageToMob(CanMessageType_t *Message)
{
  ENTER_CRITICAL_SECTION;
    
  // write DLC (Data Length Code)
  CANCDMOB = Message->Length;
  
  if (Message->Flags.Extended == 1)
  {
          PORTE^=1<<PE4;//Status LED
          // Eigentlich dürfe das Flag nie gesetzt sein..!
  }
  
  if (Message->Flags.Extended) 
  {
    // extended CAN ID
    CANCDMOB |= (1 << IDE);
    
    CANIDT4 = (uint8_t)  Message->Identifier << 3;
    
    uint32_t temp = Message->Identifier << 3;
    uint8_t *ptr = (uint8_t *) &temp;
    
    CANIDT3 = *(ptr + 1);
    CANIDT2 = *(ptr + 2);
    CANIDT1 = *(ptr + 3);
  }
  else 
  {
    // standard CAN ID
    CANIDT4 = 0;
    CANIDT3 = 0;
    CANIDT2 = (uint8_t)  Message->Identifier << 5;
    CANIDT1 = (uint16_t) Message->Identifier >> 3;
  }    

  
  if (Message->Flags.Rtr) {
    CANIDT4 |= (1<<RTRTAG);
  }
  else 
  {
    const uint8_t *p = Message->Data;
    for (uint8_t i = 0;i < Message->Length;i++) 
    {
      CANMSG = *p++;
    }
  }
  LEAVE_CRITICAL_SECTION;
}

Mir ist aufgefallen das:

  if (Message->Flags.Extended == 1)
  {
    // Eigentlich dürfe das Flag nie gesetzt sein..?!
  }

Das dass Flag gesetzt wird. ABER NICHT VON MIR!
Das heißt es muss ja irgendwas falsch laufen!

Habe die Source und den Header beigefügt. Was übersehe Ich?

[Eingast (Gast)]

Jan H. schrieb:
> Das dass Flag gesetzt wird. ABER NICHT VON MIR!
> Das heißt es muss ja irgendwas falsch laufen!

Dann hast du unter Garantie irgendwo im Code einen Pointer der wo 
rumschreibt wo er nicht darf, oder ein Array out of bounds.