2017-05-08 4 views
-2

Ich versuche, Netzwerkschnittstellennamen von Linux-Datei "/ proc/net/dev" zu erhalten. Ich habe zwei Fragen. Erstens kann ich es erhalten, indem Sie diesen Code aus der 3. Zeile bis zum Ende der Datei zu schreiben:Segmentierungsfehler wegen malloc

char buffer[100], word[10]; 
    fgets(buffer, 100, fp); 
    sscanf(buffer, "%s %*[:] %*s", word); 

aber das Problem ist, ich die Saiten mit einem Doppelpunkt erhalten (siehe: eth0: eth1 :). Wie kann ich sie so bekommen, wie ich sie auf dem Bildschirm anzeigen kann?

Die zweite Frage ist, warum ich einen Segmentation Fault auf folgenden Code erhalten:

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

#define NETFILE "/proc/net/dev" 

static char **interface_names; 

int counting_lines() 
{ 
    FILE *fp = fopen(NETFILE, "r"); 
    int i = 0; 
    char buffer[200]; 

    while (fgets(buffer, sizeof(buffer), fp) != NULL) 
    { 
     i++; 
    } 
    fclose(fp); 
    return i; 
} 

void do_malloc() 
{ 
    int i; 
    int lines = counting_lines() - 2; 

    interface_names = (char **)malloc(lines * sizeof(char *)); 

    for (i = 0; i < lines; i++) 
    { 
     interface_names[i] = (char *)malloc(10 * sizeof(char)); 
    } 
} 

void free_malloc() 
{ 
    int i; 

    for (i = 0; i < (counting_lines() - 2); i++) 
    { 
     free(interface_names[i]); 
    } 
    free(interface_names); 
} 

void get_interface_names() 
{ 
    FILE *fp = fopen(NETFILE, "r"); 
    int i = -2; 
    char buffer[100]; 

    while (!feof(fp)) 
    { 
     if (i < 0) 
     { 
      i++; 
      continue; 
     } 
     else 
     { 
      fgets(buffer, 100, fp); 
      sscanf(buffer, "%s %*[:] %*s", interface_names[i]); 
      i++; 
     } 
    } 
    fclose(fp); 
} 

int main() 
{ 
    do_malloc(); 
    get_interface_names(); 
    printf("%s\n", interface_names[0]); 
    printf("%d\n", counting_lines()); 
    free_malloc(); 
    return EXIT_SUCCESS; 
} 
+2

Eine Frage pro Frage bitte. –

+5

'while (! Feof (fp))' erkennt immer das letzte Element nicht. Sie werden mit einem Duplikat am Ende Ihrer Liste enden. –

+0

Verwenden Sie stattdessen 'strsp()'. – 0andriy

Antwort

1

Ihre sscanf Muster falsch ist. Es sollte sein:

Der Speicherplatz am Anfang überspringt alle Leerzeichen am Anfang der Zeile. Dann analysiert es alles bis zum nächsten : und setzt es in interface_names[i], mit einem Maximum von 9 Zeichen erlaubt (seit interface_names[i] ist 10 Bytes).