Ошибка сегментации MEMCPY при попытке копирования в элемент структуры - PullRequest
Новая
       23

Ошибка сегментации MEMCPY при попытке копирования в элемент структуры

1 голос
/ 28 декабря 2011

Код: ( Я пометил ошибочную строку комментарием ) 

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

typedef struct stack_node_type{
    int *nr;
    char *string;
struct stack_node_type *next;
} SNODE;

SNODE * pushStack(SNODE **stack, int *nr, char *string){
SNODE *snode=NULL;
if(nr!=NULL){
    snode = (SNODE *) malloc(sizeof(SNODE));
    aux=snode->nr;
    printf("%d\n", *nr);
    memcpy(snode->nr, nr, sizeof(int)); //THIS IS THE FAULTY LINE
    if(*(&stack)!=NULL){
        snode->next=&(**stack);
    }
    else{
        snode->next=NULL;
    }
    if(string!=NULL){
        snode->string=&(*string);
    }
}
else{
    if(string!=NULL){
        snode = (SNODE *) malloc(sizeof(SNODE));
        if(*(&stack)!=NULL){
            snode->next=&(**stack);
        }
        else{
            snode->next=NULL;
        }
        snode->string=&(*string);
    }
}
if(snode!=NULL){
    return &(*snode);
}
else{
    return &(**stack);
}
}

SNODE * popStack(SNODE **stack, SNODE *pop){
SNODE *snode=NULL;
snode=&(**stack);
if(snode!=NULL){
    if(snode->nr!=NULL){
        pop->nr=(int *) malloc(sizeof(int));
        * (pop->nr) = * (snode->nr);
    }
    if(snode->string!=NULL){
        int strdim = strlen(snode->string);
        pop->string=(char *) malloc(strdim*sizeof(char));
        strcpy(pop->string, snode->string);
    }
    SNODE *to_del=snode;
    snode=snode->next;
    free(to_del);
}
return &(*snode);
}

int main()
{
SNODE *stack=NULL;
SNODE pop;
int nr;
nr=123;
stack=pushStack(&stack, &nr, "banane");
nr=819;
stack=pushStack(&stack, &nr, "portocale");
while(stack!=NULL){
    stack=popStack(&stack, &pop);
    printf("POP: %d, \"%s\"\n", *(pop.nr), pop.string);
}
    return 0;
}

Восстановление неисправной линии :

memcpy (snode-> nr, nr, sizeof (int)); // ЭТО НЕПРАВИЛЬНАЯ ЛИНИЯ

Memcpy должен треснуть, когда принимается недоступная память или блоки памяти источника и приемника перекрываются, поэтому, насколько я понимаю, ни одна из этих проблем не кажется действительной. Почему он трескается?

1 Ответ

4 голосов
/ 28 декабря 2011

Вы выделили память для структуры, но не выделили для самих членов.

Попробуйте: 

snode = (SNODE *) malloc(sizeof(SNODE));
snode->nr = malloc(sizeof(int));

Но на вашем месте я бы изменил структуру: 

struct stack_node_type{
    int nr;
    char *string;
};

memcpy(&snode->nr, nr, sizeof(nr));
...