2

Tengo esta pieza de código C (Para plataformas MSP430, compilador cl430):

void function(uint8_t * x){
    // Esta variable está definida en el archivo linker file como "X_ADDR = 0xE000;"
    extern uint32_t X_ADDR; 
    uint16_t i = 0;
    uint16_t size = 10;
    uint32_t dst_addr = (uint32_t) &X_ADDR; 

    for (i=0; i < size; i++){
         *((uint8_t *) (dst_addr+i)) = *(x+i); // <-- Esta línea muestra la advertencia
}

Lo que entiendo es esto, aunque creo que estoy equivocado aquí:

*((uint8_t *) (dst_addr+i)) = *(x + i);
                |       |       |   |
                V       V       V   V
*((uint8_t *) (u32  + u16)) = *(u8*+u16);
       *((uint8_t *) (u32)) = *(u8*);
                     *(u8*) = u8;
                         u8 = u8;

Esta plataforma es de 16-bits pero soporta un modo extendido de direccionamiento de 20-bits.

¿Alguna opinión al respecto? y ¿Cómo debería ser escrito?

Gracias de antemano

2 respuestas 2

2

Los compiladores advierten de la posiblemente no intencionada pérdida de precisión en la conversión:

clang

cast to uint8_t * (aka unsigned char *) from smaller integer type unsigned int [-Wint-to-pointer-cast]

gcc

cast to pointer from integer of different size [-Wint-to-pointer-cast]


Los punteros suelen tener el tamaño de la palabra de procesador. En arquitecturas de 64 bits será un entero de 64 bits ([u]int64_t) que será más grande que el tamaño de int (que habitualmente es representado por [u]int32_t).

Por eso da una alarma al convertir tipos de diferentes tamaños, compruébalos:

#include <stdint.h>

#define T(X) printf("Tamanyo de " #X " = %d\n", sizeof(X))

int main()
{
    T(uint8_t);   // Tamanyo de uint8_t = 1
    T(uint16_t);  // Tamanyo de uint16_t = 2
    T(uint32_t);  // Tamanyo de uint32_t = 4

    T(uint8_t*);  // Tamanyo de uint8_t* = 8
    T(uint16_t*); // Tamanyo de uint16_t* = 8
    T(uint32_t*); // Tamanyo de uint32_t* = 8
}

Para evitar el problema haz que dst_addr sea un puntero, que al fin y al cabo es lo que quieres:

void function(uint8_t * x){
    uint16_t i = 0;
    uint16_t size = 10;
    uint32_t* dst_addr = 0; // dummy value
//  ^^^^^^^^^ <--- Puntero

    for (i=0; i < size; i++){
         *((uint8_t *) (dst_addr+i)) = *(x+i);
    }
}
2
  • Hola, gracias por la respuesta. He añadido un poco mas de detalle. En cuanto al tratar la variable como puntero, prefiero hacer la aritmética con enteros ya que al final necesito escribir a nivel de bytes (8 bits). Aquí está la pregunta en SO si quieres revisar la discusión allí. el 7 nov. 2019 a las 23:10
  • @gustavovelascoh los detalles que has añadido no cambian la respuesta. Si quieres quitar la alarma usa punteros. Si estás usando enteros para tratarlos como punteros, obtendrás alarmas, independientemente de lo que prefieras. el 8 nov. 2019 a las 7:18
0

El problema es que el código está pensado para un modelo de memoria extendida, con punteros de 20-bits (uint32_t), pero este no estaba activado y usaba el modo de punteros de 16 bits.

Activándolo o cambiando uint32_t dst_addr = (uint32_t) &X_ADDR; a uint16_t o uintptr_t, ya no genera el mensaje de advertencia.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.