0

Buenas. Mi objetivo es construir un array de estructuras que me relacionan un comando con un puntero a función, para lo cual he definido una estructura de la siguiente manera:

typedef struct function_pointers_array
{
    unsigned char * command_code;
    unsigned char * (*function_pointer)(unsigned char *, int);
} function_pointers_array_t;

Y inicializo el array como sigue:

function_pointers_array_t f_ptr_array[] = {
    { "0601", cpuUnitStatusRead },
    /*{...},
    {...},
    ...*/
};

Mi duda y el problema que tengo es porqué cuando hago lo siguiente obtengo una violación de segmento:

if (strcmp (f_ptr_array[i].command_code, cmd_code) == 0) //segmentation fault

Siendo cmd_code del tipo static unsigned char. El problema según he comprobado está en f_ptr_array[i].command_code, ¿qué hay de problemático en esto?

Muchas gracias de antemano.


En respuesta a @eferion:

#include <stdio.h>

unsigned char * cpuUnitStatusRead();
unsigned char * memoryAreaRead();

typedef struct function_pointers_array
{
    unsigned char * command_code;
    unsigned char * (*function_pointer)(unsigned char *, int);
} function_pointers_array_t;

function_pointers_array_t f_ptr_array[] = {
    { "0601", cpuUnitStatusRead },
    { "0101", memoryAreaRead },
};

int main()
{
    int i;
    char cmd_code[]="0601";

    for (i=0; i<2; i++)
    {
        if (strcmp (f_ptr_array[i].command_code, cmd_code) == 0)
        {
            printf("\nOk");
        }
    }

}

unsigned char * cpuUnitStatusRead()
{
    //
}

unsigned char * memoryAreaRead()
{
    //
}
17
  • Si pones un ejemplo mínimo que reproduzca el problema será más sencillo ayudarte.
    – eferion
    el 27 dic. 2016 a las 11:26
  • El problema podría estar en, por ejemplo, que el índice i es igual o superior al número de elementos, lo que haría que el código intentase acceder a memoria fuera del arreglo
    – eferion
    el 27 dic. 2016 a las 11:28
  • No sé realmente cómo facilitar un ejemplo sin proveer todo el código. Solo decir que no es problema del índice como comentas, de hecho el mismo problema ocurre si utilizo el índice 0.
    – elis
    el 27 dic. 2016 a las 12:31
  • Por otro lado, dejo lo que me muestra el depurador: Program received signal SIGSEGV, Segmentation fault. [Cambiando a Thread 0x7ffff6ff4700 (LWP 2230)] __memcpy_sse2_unaligned () at ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S:140 140 ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S: No existe el archivo o el directorio.
    – elis
    el 27 dic. 2016 a las 12:33
  • Antes del srtcmp imprime los valores, puede ser que tengas un nulo en alguno de los dos campos el 27 dic. 2016 a las 12:50

2 respuestas 2

2

Esto debería ser un comentario, pero no es viable ponerlo como tal por su extensión y contenido.

memcpy(cmd_code, command+offset, 2);

Aquí tenemos el problema de que no sabemos qué contiene command ni offset. Aún así asumiremos que está todo bien.

Nota eso sí que estás copiando dos bytes lo cual, seguramente, no incluye el byte de terminación de cadena. Este detalle es importante porque aquí:

strcmp (f_ptr_array[i].command_code, cmd_code)

Estas comparando dos cadenas con strcmp. Esta función compara bytes hasta que encuentra una diferencia o un carácter nulo. En tu caso una de las dos cadenas únicamente tiene 2 bytes asignados y carece de terminador de cadena.

Cuando te pedimos que pongas un ejemplo mínimo y completo nos referimos a un código que podamos compilar y ejecutar sin problemas.

4
  • De acuerdo. He realizado un pequeño programa listo para ejecutar (disponible en la pregunta editada) que parece que no da ningún problema, sin embargo mi código no es diferente a éste y el problema persiste por más que elimines el bucle o accedas al elemento 0 del array. Seguiré probando, muchas gracias por la ayuda
    – elis
    el 28 dic. 2016 a las 8:52
  • Por otro lado también comentar que el valor de cmd_code lo he asignado manualmente en este caso ya que en mi código lo recibe de una cadena recibida por un socket. Sin embargo, he comprobado que es correcto su valor y no influye en el problema
    – elis
    el 28 dic. 2016 a las 9:05
  • 1
    Si el problema persiste posiblemente se deba a que el error no está donde tú crees. Ahí está la gracia de poner un ejemplo mínimo que reproduzca el problema. Una cosa es dónde casca la aplicación y otra el origen del problema... No tienen por qué ser la misma cosa
    – eferion
    el 28 dic. 2016 a las 11:55
  • Eso es lo que parece... Me apunto la recomendación para el futuro
    – elis
    el 28 dic. 2016 a las 17:53
0

Aquí puedes comprobarlo: http://rextester.com/PIF91009

Independientemente del tipo de parametros lo unico que debes indicar del puntero a una funcion, es el tipo de valor devuelto, en tu caso:

unsigned char * (*funcion)();

Aunque no veo que tu funcion devuelva ningún puntero.

#include  <stdio.h>

unsigned char a = 'A';
unsigned char b = 'B';

static unsigned char *foo1()
{
    return &a;
}

static unsigned char *foo2(int p1, int p2)
{
    return &b;
}


typedef struct mystruct
{
    int id;
    unsigned char * (*funcion)(); 
} pfunction;    

int main(void)
{    
    pfunction s []  = {{12, &foo1, "n1"},{13, &foo2, "n2"}};

    printf ("El array s tiene %d elementos\n", sizeof(s) / sizeof(pfunction));

    unsigned char *a1 = s[0].funcion();
    printf ("foo1() = %c   nombre = %s\n", *a1, s[0].nombre);

    unsigned char *b1 = s[1].funcion(10,20);
    printf ("foo2() = %c   nombre = %s\n", *b1, s[1].nombre);
}

Resultado:

El array s tiene 2 elementos 
foo1() = A   nombre = n1 
foo2() = B   nombre = n2

Si te fijas en este ejemplo, foo1 y foo2 tienen diferentes parametros, pero a tu puntero esto le da igual. Le basta con saber que es un puntero a una funcion que devuelve un unsigned char*

Para assignar los valores, prefiero usar la dirección de la función, aunque no es necesario queda mucho más claro:

pfunction s []  = {{12, &foo1},{13, &foo2}};

Espero que esto te sirva.

Y por si te sirve de ayuda, te dejo un enlace a la clase en c# para conectar con un PLC, que tengo publicada en CodeProject.

5
  • Gracias por el aporte. Teniendo ese código en cuenta parece que no hay nada erróneo en mi código. Mi problema realmente está cuando intento acceder a lo que en este caso sería (usando un índice cualquiera) s[0].id
    – elis
    el 28 dic. 2016 a las 9:13
  • Estas usando este bucle for (i=0; i<2; i++)? O solo es para el ejemplo?
    – McNets
    el 28 dic. 2016 a las 9:57
  • Por que si pegas tu codigo en rextester funciona correctamente.
    – McNets
    el 28 dic. 2016 a las 9:59
  • Creo que deberias publicar el codigo real, la assignacion de punteros y el bucle que usas pafa la comparacion.
    – McNets
    el 28 dic. 2016 a las 17:51
  • Acabo de dar con el problema, venía de otro lugar a pesar de que la ejecución se detenía ahí. Efectivamente el código del ejemplo es correcto
    – elis
    el 28 dic. 2016 a las 17:58

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

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