conversione non valida da 'void *' a 'nodo *' [-fpermissive]

Question

Ho un programma C che genera un errore:

invalid conversion from 'void*' to 'node*' [-fpermissive] 

Ecco il mio codice:

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

struct node 
{ 
    int data; 
    struct node* next; 
}; 

struct node* onetwothree(); 

int main() 
{ 
    struct node* ptr; 
    ptr = onetwothree(); 
    return 0; 
} 

struct node* onetwothree() 
{ 
    struct node* head; 
    struct node* temp; 
    head = malloc(sizeof(struct node)); 
    temp = head; 
    for(int i=1; i<=3; i++) 
    { 
     temp->data = i; 
     if(i<3) 
      temp=temp->next; 
     else 
      temp->next = NULL; 
    } 
    return head; 
} 

Che cosa sono io fare male?

Answer 1

In C, un void* è implicito convertibile in T* dove T è di qualsiasi tipo. Dalla sezione 6.3.2.3 Puntatori dello standard C99:

A pointer to void may be converted to or from a pointer to any incomplete or object type. A pointer to any incomplete or object type may be converted to a pointer to void and back again; the result shall compare equal to the original pointer.

malloc() restituisce un void* ed è assegnabile, senza fusione a head, un struct node*. Questo non è vero in C++, quindi ho il sospetto che un compilatore C++ venga usato per compilare questo codice C.

Ad esempio:

#include <stdlib.h> 

int main() 
{ 
    int* i = malloc(sizeof(*i)); 
    return 0; 
} 

se compilato con:

gcc -Wall -Werror -pedantic -std=c99 -pthread main.c -o main

emette alcun errore. Se compilato con:

g++ -Wall -Werror -pedantic -std=c++11 -pthread main.cpp -o main

emette:

main.cpp: In function 'int main()': main.cpp:5:31: error: invalid conversion from 'void*' to 'int*' [-fpermissive]

---

Inoltre, la funzione onetwothree() non alloca memoria correttamente. Si assegna un solo struct node:

head = malloc(sizeof(struct node)); 

e poi, eventualmente, dereferenziazioni head->next->next che è un comportamento indefinito. È necessario un singolo malloc() per ogni struct node. Ricorda di free() che cosa era malloc() d.

Answer 2

Stai avendo questo avviso/errore perché si utilizza malloc (che restituisce un void*) per inizializzare una struttura di tipo node* senza fare un cast esplicito.

per sbarazzarsi di questo errore si potrebbe modificare il codice in questo modo:

head = (struct node *)malloc(sizeof(struct node)); 

oppure si potrebbe anche aggiungere il flag "-fpermissive" per il compilatore che verrà poi ignorare questi errori.

EDIT: Ma sì, non ho pensato al fatto che questo non dovrebbe accadere in un compilatore C in ogni modo