C++: erreur de segmentation sous Linux OS

Question

Im essayant de compiler et d'exécuter ce petit code C++ en utilisant g ++ 5.1, il est compilé correctement, quand je l'exécute sur linux j'obtiens ce message d'erreur: "Segmentation fault (core dumped)".

Mais le même correctement code exécuté sur Mac OS X, mais pas sur linux:

#include <iostream> 
#include <string.h> 
#include <stdlib.h> 
#include <string.h> 
using namespace std; 

struct node { 
std::string data; 
}; 

int main() { 
    struct node * node = (struct node *) 
    malloc(sizeof(struct node)); 

    node->data.assign("string"); 
    // node->data = "string" --> same issue 

    return 0; 
} 

i essayé simple assigne (node-> data = "string"), mais je suis le même problème Toute aide s'il vous plaît!

Answer 1

Vous ne pouvez pas malloc une chaîne C++. Vous devriez utiliser correctement new et delete, au moins, pour que les constructeurs soient invoqués. Arrêtez d'utiliser C en C++.

Idéalement, vous n'utiliserez même pas new; Il suffit d'avoir un objet normal avec une durée de stockage automatique ou, si vous avez désespérément besoin d'une allocation dynamique, std::make_unique.

La gestion manuelle de la mémoire n'est pas nécessaire en 2016.

Answer 2

Avec C++, oubliez malloc(). Si vous souhaitez allouer un objet new utiliser:

node * n = new node; // or if your variable should be called node 
         // you'd need new struct node to disambiguate 

Le problème avec malloc() est qu'il alloue seulement la mémoire non initialisée. Il n'assure pas la sémantique C++ de la création d'objet. Ainsi, la chaîne à l'intérieur de votre noeud n'est pas initialisée à un état valide. Cela provoque l'affectation de cette chaîne à UB.

Si vous avez vraiment besoin d'utiliser malloc() en C++, vous aurez besoin d'utiliser un placement new après pour initialiser l'objet à un état valide (online demo).

void *p = malloc(sizeof(node)); // not so a good idea ! 
node *n2 = new (p)node;   // but ok, it's feasible.