[C/C++] Segmentation Fault

Smokers · 1. Juni 2010

Segmentation Fault

Ja aso ich hab eigentlich nur ne kurze Frage,...

ich hab halt wie immer unter Windows programmiert muss das Programm allerdings in Linux vorstellen.
Die typischen Fehler umgehe ich nun inzwischen schon relativ gut, allerdgins finde ich den Fehler der mir unter Linux nen Segmentation Fault bringt nicht....

Code:

int main(int argc, char** argv) {
 FILE *fp;
 char* fname;
 char key = '\0';

 if (argc > 1) {
 fname = argv[1];
 } else {
 readFileName(fname);
 }
 ............
 ............
 ............
 ............
}

void readFileName(char* fname) {
 do {
 printf("Bitte geben Sie den Dateinamen ein!\n");
 scanf("%s60", fname);
 } while (strlen(fname) < 1);
 fflush(stdin);
}

Der Fehler wird nur produziert wenn ich per Kommandozile kein File angebe, also muss der Fehler im Lesen oder Verarbeiten des Filenames liegen.

Unter Windows funktioniert es, und das es " scanf("%s60", &fname);" liegen kann , ist es nicht. Da bekomme ich dann unter Windows schon nen Absturz ( komme mit den Zeigern öfter durcheinander darum)

Jem ne Idee?

Anzeige

Murdoc · 1. Juni 2010

AW: Segmentation Fault

deklariere fname mit NULL und teste es nochmal.
zudem müsstest du dem pointer noch speicher reservieren.

ps: mitr &fname übergibst du nen pointer zu nem pointer

Smokers · 1. Juni 2010

AW: Segmentation Fault

Code:

char* fname = NULL;

Bekomme ich nach meinem Enter bei der Eingabe unter Windows nen Fehler.
Also soltle auch nen Segmentation Fault sein, nur Windows ist weniger aufschlussreich und beendet die exe dann einfach.

//////
Also danke an Murdoc , wurde mal im ICQ gelöst mit :

Code:

/*
 * File: main.c
 * Author: Smoki
 *
 * Created on 29. Mai 2010, 19:15
 */

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

/*
 *
 */
#define CLEAR_SCREEN "clear"


int main(int argc, char** argv) {
 FILE *fp;
 char *fname;
 char key = '\0';

 if (argc > 1) {
 fname = argv[1];
 } else {
 fname = (char *) malloc(sizeof (char) * 61);
 fname[60] = '\0';
 readFileName(fname);
 }
.......................................
.......................................
}


void readFileName(char* fname) {
 do {
 printf("Bitte geben Sie den Dateinamen ein!\n");
 scanf("%s60", fname);
 } while (strlen(fname) < 1);
 fflush(stdin);
}

Hie rnochmal eine Reaktion von : RemaL

Hey.

Dein Code ist verbesserungswürdig und da der Thread schon zu ist schreibe ich dir mal eine PN.

Anstelle von scanf() ist für Strings fgets() die bessere und, was noch viel wichtiger ist, sicherer.

Zudem ist es quatsch für eine konstante Anzahl an Zeichen speicher zu reservieren, nutze stattdessen doch einfach ein Array!

char *fname; -> char fname[61];

Mit 0'llen füllen kannst du direkt bei der Initialisierung per Zuweisung

Code:

char fname[61] = {0};

oder per memset()

Code:

memset(fname, 0, 61);

Für den Fall das alle Parameter angegeben sind kannst du nun auch strncpy() verwenden:
Code:

strncpy(fname, argv[1], 60); // 60 Zeichen kopieren, danach Schluss da sonst das Terminierungszeichen entfernt würde (bei mehr als 60 Zeichen im Parameter)

Sollte readFileName() eine NULL als Parameter bekommen, solltest du auch für einen solchen Fall gewappnet sein, und den Parameter checken und gegebenfalls die weitere Ausführung der Funktion unterbinden.

fflush() sollte zwar nicht auf stdin angewendet werden, dennoch merke ich das du noch in den Startlöchern steckst und daher ist das schon in Ordnung. Jedoch will ich ich dir eine gute Möglichkeit den Input-Stream zu leeren nicht vorenthalten:

Und zwar solltest du einfach solange aus dem Stream lesen bis nicht's mehr drin ist:

Code:

while ( getchar() != EOF );

Viel Erfolg weiterhin!

Grüße.

Klicke in dieses Feld, um es in vollständiger Größe anzuzeigen.

Nützliche Suchen

[C/C++] Segmentation Fault

Videos zum Themenbereich