1

Estou tentando alocar dinamicamente, porém está aparecendo o erro:

Error in ./nome.c; invalid pointer : 0x0000000001c18834

Meu código:

// FUNCAO // 
char *nome(text[])  
{    
    int n3 = 0;  
    int n2 = 0;     
    char *n = NULL;  

    while((n2 = getchar()) != '\0' && n2 != EOF)  
    {  
        n3++;  
        n = realloc(n, n3*sizeof(char));  
        if(n == NULL)  
        {  
            puts("Erro ao realocar!");  
            exit(0);  
            free(n);  
        }   
        *(n+(n3-1)) = n2;   
    }  
    *(n+n3) = '\0';  
    return n;
}

// USANDO //   
int main(void)  
{  
    char *name = nome("Nome:");  
    while(*name != '\0')  
    printf("%c", *name++);    
    free(name); // aqui ta o problema, sem o free roda normal agora com o free me da esse erro
}        
4
  • 1
    Esse código nem compila, tem erros de sintaxe, então fica complicado até falar alguma coisa. Mas experimentou usar free(name). Não tem porque derreferenciá-lo.
    – Maniero
    Commented 19/06/2017 às 20:13
  • nao sei editar o texto aqui da ultima vez alguem edito pra min
    – LavaiBala
    Commented 19/06/2017 às 20:15
  • Para editar sua pergunta clique em editar. Commented 19/06/2017 às 20:18
  • testa ai alguem com o free e sem o free
    – LavaiBala
    Commented 19/06/2017 às 20:20

1 Resposta 1

2

O código tem alguns problemas, inclusive não está legível. Simplifiquei um pouco. O problema mais sério mesmo é que guarda em name o ponteiro para o local alocado. Ele não pode ser modificado para que depois possa ser dado o free no local coreto. Quando o incrementa vai tentar liberar memória em lugar que não houve alocação e gera o erro. O correto é criar uma variável de iteração independente e ir incrementando ela, assim não mexe me name.

Não vi se tem outros problemas.

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

char *nome(char text[]) {    
    int n3 = 0;
    int n2 = 0;
    char *n = NULL;
    while((n2 = getchar()) != '\0' && n2 != EOF) {
        n = realloc(n, ++n3);
        if (n == NULL) {
            puts("Erro ao realocar!");
            exit(0);
        }
        *(n + (n3 - 1)) = n2;   
    }
    *(n + n3) = '\0';  
    return n;
}   

int main(void) {  
    char *name = nome("Nome:");
    char *iterador = name;
    while(*iterador != '\0') printf("%c", *iterador++);
    printf("%s", name); //bem mais simples, certo?
    free(name);
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

5
  • deu certo usar o iterador vlw amigo, e sobre o codigo não está legível é porque eu não sei editar aqui no stackoverflow
    – LavaiBala
    Commented 19/06/2017 às 20:36
  • Não, o problema não é esse, inclusive o meu ainda não está tão legível, porque não quis mexer demais no seu código. Note que usando o printf() nem precisaria do iterador.
    – Maniero
    Commented 19/06/2017 às 20:37
  • eu nao copiei e colei eu escrevi tudo aqui
    – LavaiBala
    Commented 19/06/2017 às 20:38
  • Primeiro não deveria ter feito isto, segundo, a ilegibilidade está ano código, não da digitação.
    – Maniero
    Commented 19/06/2017 às 20:39
  • pode ser então que meu código não esteja legivel, vlw pelo toque vou tentar melhorar nesse aspecto, mais o código já esta rodando obrigado a todos que ajudarão
    – LavaiBala
    Commented 19/06/2017 às 20:44

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .