Início / coding / Perguntas / 78849728
Accepted
user180574
Asked: 2024-08-09 01:34:40 +0800 CST

C: por que recebo falha de segmentação?

  • 772

Eu tenho o seguinte código. Chamar f1 gera segfault, mas não f2.

#include <stdlib.h>
#include <stdio.h>

void f1(unsigned char** arr)
{
   unsigned char* p = *arr;
   *p = 'h';
   p++;
   *p = '\0';
}

void f2(unsigned char* arr)
{
   unsigned char* p = arr;
   *p = 'h';
   p++;
   *p = '\0';
}

int main()
{
   unsigned char a[31];
   f1((unsigned char**)&a);
   //f2(a);
   printf("%s\n", a);
}
c

1 respostas

  1. Best Answer

    unsigned char**é um ponteiro para um ponteiro para um unsigned char.

    &aretorna o endereço do array a[]. O tipo que &aretorna é unsigned char (*)[31], não unsigned char** , e é por isso que você não pode simplesmente chamar f1(a)which é uma incompatibilidade de tipo. Preste atenção nesse erro, é importante.

    &anão retorna o endereço de um ponteiro, mas você está usando um typecast para induzir o compilador a pensar que sim, portanto, você está forçando f1()o acesso a um endereço de memória inválido por meio de um ponteiro inválido, daí o segfault.

    IOW, você está pegando um ponteiro para array e convertendo-o em um ponteiro para ponteiro , o que não é. Dentro de f1(), a *arrdesreferência espera produzir um unsigned char*ponteiro para um válido unsigned char, mas esse não é realmente o caso. arrestá apontando para a[]si mesmo e, portanto, a leitura *arrlerá e interpretará mal os primeiros 4 bytes (em uma compilação de 32 bits) ou 8 bytes (em uma compilação de 64 bits) como a[]sendo o valor de um ponteiro, o que não é. Então a desreferenciação desse ponteiro falha.

    Não use typecasts cegamente apenas para silenciar erros do compilador. Eles são erros por um motivo. Typecasts são úteis em certos casos, mas este não é um deles. A maneira corretaf1() de chamar a[]no seu exemplo é assim:

    unsigned char a[31];
unsigned char *ptr = a; // aka: = &a[0]
f1(&ptr);

    Alternativamente, mude f1()para aceitar um ponteiro para array em vez de um ponteiro para ponteiro:

    #include <stdlib.h>
#include <stdio.h>

void f1(unsigned char (*arr)[31])
{
   unsigned char* p = *arr;
   *p = 'h';
   p++;
   *p = '\0';
}

int main()
{
   unsigned char a[31];
   f1(&a);
   printf("%s\n", a);
}

    Quanto ao motivo pelo qual a chamada f2(a)funciona como está, é porque um array decai em um ponteiro para seu primeiro elemento e f2()está usando esse ponteiro corretamente. Esse não é o caso quando você está ligando f1(&a).

    • 5

relate perguntas