Malloc und scanf

  char *toParseStr = (char*)malloc(10);
  printf("Enter string here: ");
  scanf("%s",toParseStr);
  printf("%s",toParseStr);
  free(toParseStr);

Erstens, die Zeichenfolge in scanf ist, gibt die input es geht, zu erhalten. Um die Anzeige einer Zeichenkette vor der Annahme von Tastatureingaben verwenden printf wie gezeigt.

Zweitens, Sie brauchen nicht zu dereferenzieren toParseStr da es zeigt, um ein Zeichen-array der Größe 10, wie Sie zugewiesen mit malloc. Wenn Sie wurden mit einer Funktion, die würde, zeigen Sie es auf einem anderen Speicher-Ort, dann &toParseStr erforderlich ist.

Angenommen, Sie wollten eine Funktion schreiben, die Speicher reservieren. Dann müssten Sie &toParseStr da änderst du den Inhalt der pointer-variable (die eine Adresse im Speicher --- Sie selbst sehen können, durch den Druck seiner Inhalte).

void AllocateString(char ** ptr_string, const int n)
{
    *ptr_string = (char*)malloc(n)
}

Wie Sie sehen können, es akzeptiert char ** ptr_string das liest sich wie ein Zeiger speichert die Speicherposition einen Zeiger, die wird speichern die Speicher-Adresse (nach der malloc Betrieb) das erste byte von einem zugewiesenen block n bytes (derzeit sind es einige Müll-Speicher-Adresse, da es nicht initialisiert ist).

int main(int argc, char *argv[])
{
  char *toParseStr;
  const int n = 10;
  printf("Garbage: %p\n",toParseStr);
  AllocateString(&toParseStr,n);
  printf("Address of the first element of a contiguous array of %d bytes: %p\n",n,toParseStr);

  printf("Enter string here: ");
  scanf("%s",toParseStr);
  printf("%s",toParseStr);
  free(toParseStr);

  return 0;
}

Drittens, es wird empfohlen, um Speicher frei, die Sie zuordnen. Obwohl dies Ihr ganzes Programm, und dieser Speicher wird freigegeben, wenn das Programm beendet wird, es immer noch gute Praxis.

InformationsquelleAutor Jacob

Mit scanf() (oder fscanf() auf Daten, die du nicht kontrollierst), standardmäßig "%s" Bezeichner ist eine fast bestimmte Art und Weise, um sich selbst in Probleme mit buffer-overflows.

Das klassische Beispiel ist, dass ich geben Sie die Zeichenfolge "Dieser string ist so mehr als 10 Zeichen" in Ihrem Programm, das chaos wird Folgen, Katzen und Hunde werden beginnen, zusammen zu schlafen und eine Nackte Singularität kann auch angezeigt werden und verbrauchen die Erde (die meisten Leute nur sagen "Undefiniertes Verhalten", aber ich denke meine Beschreibung ist besser).

Ich aktiv raten von der Verwendung von Funktionen, die nicht in Schutz. Ich möchte Sie bitten (vor allem als Neuling auf C) um fgets() Lesen Ihre Eingabe, da Sie kontrollieren können, buffer overflows mit die es viel einfacher, und es ist mehr geeignet, einfache Online-Eingabe als scanf().

Sobald Sie eine Linie, Sie können dann rufen sscanf() auf es zu Inhalt Ihres Herzens, die, übrigens, Sie brauchen nicht zu tun, in diesem speziellen Fall, da du bist nur immer ein raw-string sowieso.

Würde ich verwenden:

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

#define BUFFSZ 10

int main(int argc, char *argv[]) {
  char *toParseStr = malloc(BUFFSZ+2);
  if (toParseStr == NULL) {
      printf ("Could not allocate memory!\n");
      return 1;
  }
  printf ("Enter a string: ");
  if (fgets (toParseStr, BUFFSZ+2, stdin) == NULL) {
      printf ("\nGot end of file!\n");
      return 1;
  }
  printf("Your string was: %s",toParseStr);
  if (toParseStr[strlen (toParseStr) - 1] != '\n') {
      printf ("\nIn addition, your string was too long!\n");
  }
  free (toParseStr);
  return 0;
}

InformationsquelleAutor paxdiablo

Ersten, die Fehler, die halten Sie Ihr Programm von der Arbeit: scanf(3) nimmt ein format-string genau wie printf(3), nicht um eine Zeichenfolge zu drucken für die Benutzer. Zweitens, Sie waren übergeben der Adresse des Zeigers toParseStr, anstatt der Zeiger toParseStr.

Ich habe auch entfernt die unnötige Umwandlung von Ihrem Anruf malloc(3).

Eine Verbesserung, die Ihr Programm noch benötigt, ist die Verwendung scanf(3)'s a option, um Speicher für Sie-so dass einige joker setzen zehn Zeichen in dein string nicht starten stampfenden auf unabhängigen Speicher. (Ja, die C mal jemandem überschreiben, der fast den gesamten Adressraum mit diesem Programm, wie geschrieben. Riesen-Sicherheitslücke. 🙂

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

int main(int argc, char *argv[])
{
  char *toParseStr = malloc(10);
  printf("Enter a short string: ");
  scanf("%s",toParseStr);
  printf("%s\n",toParseStr);
  return 0;
}

InformationsquelleAutor sarnold