Pourquoi mon programme génère-t-il une erreur de segmentation lors de l'affectation de la mémoire à un double pointeur C

Question

Pourquoi ce programme entraîne-t-il une erreur de segmentation? J'essaie d'avoir un tableau de pointeurs qui est dynamiquement alloué de la mémoire afin que je puisse avoir un tableau de chaînes.

J'ai cherché des problèmes similaires comme How to pass a double pointer to a function without segmentation fault C language

S'il vous plaît expliquer pourquoi il SEG-formation de failles

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

void mem_alloc(char* p, char** dp); 

int entries = 0; 
int mem_allocated = 0; 

int main() { 

    char* p = "ksdfahj93qhf9"; 
    char* p1 = "siodfnrieopq"; 
    char* p2 = "erf9ih94gri9g"; 

    char** dp = NULL; 

    mem_alloc(p, dp); 
    mem_alloc(p1, dp); 
    mem_alloc(p2, dp); 

    for(int i = 0; i < entries; i++) { 

     printf("%s", dp[i]); 
    } 
} 
void mem_alloc(char *p, char** dp) { 
    if(entries == mem_allocated) 
     if(mem_allocated == 0) 
      mem_allocated = 3; 
    void** temp = realloc(dp, mem_allocated * (sizeof(p))); 
    if(!temp) 
     perror("Memory allocation failed!"); 

    dp = (char**) temp; 
    strcpy(dp[entries++], p); 

} 

Answer 1

Dans votre fonction mem_alloc vous modifiez le paramètre de la fonction dp. Cette modification n'est pas visible en dehors de la fonction. Par conséquent, le dp dans main ne change jamais et est toujours défini sur NULL.

Vous devez transmettre l'adresse de cette variable à la fonction, puis dans la fonction, vous déréférencer ce pointeur pour le modifier.

Vous fonctionnez devient:

void mem_alloc(char *p, char ***dp) { 
    if(entries == mem_allocated) 
     if(mem_allocated == 0) 
      mem_allocated = 3; 
    char **temp = realloc(*dp, mem_allocated * (sizeof(p))); 
    if(!temp) 
     perror("Memory allocation failed!"); 

    *dp = temp; 
    (*dp)[entries++] = strdup(p); // space also needs to be allocated for the new string 
} 

Et vous l'appelez comme ceci:

mem_alloc(p, &dp); 

Answer 2

Deux erreurs. Le premier est celui mentionné par dbush. Deuxièmement, vous n'allouez pas d'espace pour votre chaîne avant de la copier. Vous pouvez utiliser strndup() au lieu de strcpy().