GCC: warning type-punned pointer

[Ralph (Gast)]

Wie muss die unten stehende C-Zeile *1 geschrieben werden, damit die 
folgende Warnung nicht mehr Auftritt:

stack.c:73:5: warning: dereferencing type-punned pointer will break 
strict-aliasing rules [-Wstrict-aliasing]

 *1

(*((unsigned long*)&myip[0])) = get_eeprom_value(IP_EEPROM_STORE,MYIP);

'myip' und 'get_eeprom_value' sind wie folgt festgelegt 
(Webserver-Quelltext von U. Radig):

unsigned char myip[4];

//----------------------------------------------------------------------------
//
unsigned long get_eeprom_value (unsigned int eeprom_adresse,unsigned long default_value)
{
  unsigned char value[4];
  
  for (unsigned char count = 0; count<4;count++)
  {
    eeprom_busy_wait ();  
    value[count] = eeprom_read_byte((unsigned char *)(eeprom_adresse + count));
  }

  //Ist der EEPROM Inhalt leer (0xFF) oder Null (0x00) ?
  if (((*((unsigned long*)&value[0])) == 0xFFFFFFFF) || ((*((unsigned long*)&value[0])) == 0x00000000))
  {
    return(default_value);
  }
  return((*((unsigned long*)&value[0])));
}

[Jörg W. (dl8dtl) (Moderator)]

Ich würde statt der wilden Herumcasterei eine union benutzen.

Alternativ sollte mal jemand Herrn Radig mitteilen, dass es auch eine
Funktion eeprom_read_dword() gibt.

[Oliver (Gast)]

Tja, du könntest entweder mit -WNostrict-aliasing compilieren, dann ist 
die Warnung weg ;), oder du könntest nachlesen, was die Warnung dir 
eigentlich sagen möchte, und dann da mal gründlich drüber nachdenken.

Oliver

[Karl H. (kbuchegg)]

Probier mal

unsigned long get_eeprom_value (unsigned int eeprom_adresse, unsigned long default_value)
{
  unsigned long value;
  
  eeprom_read_block( &value, (unsigned char *)eeprom_adresse, sizeof( value ) );

  if ( value == 0xFFFFFFFF) || value == 0x00000000 )
    return default_value;

  return value;
}

Edit:
wobei mir der Cast nach unsigned char * auch noch nicht gefällt. Da 
frage ich mich, warum eeprom_adresse nicht gleich ein derartige Pointer 
in der Argumentliste ist.

[Ralph (Gast)]

@ Jörg & Karl

Danke für die Infos.

[holger (Gast)]

Ich hab mir einfach eine Funktion geschrieben die den Kram
umrechnet und dann alle Stellen mit den Warnings ersetzt.

//Converts uint8_t array IP to uint32_t
uint32_t ToIP(uint8_t *val)
{
  return ( (uint32_t)val[0] + ((uint32_t)val[1] << 8) + ((uint32_t)val[2] << 16) + ((uint32_t)val[3] << 24) );
}

Danach sieht der Code auch schön leserlich aus. Beispiel:

  if( ip->IP_Destaddr == ToIP(myip) )  // if my IP address