Malloc-Fehler: falsche Prüfsumme für befreite Objekt

Arbeite ich an der Implementierung Schwanz für eine Zuordnung. Habe ich es richtig funktioniert aber ich scheine immer einen Fehler aus frei zu zufälligen Zeiten.

Kann ich nicht sehen, es aufspüren, um ein Muster oder etwas anderes, es ist konsistent.

Zum Beispiel, wenn ich rufe mein Programm als "Schwanz -24 test.in" ich würde die die die falsche checksum error in der gleichen Zeile auf mehreren läuft. Jedoch mit verschiedenen Dateien und auch eine unterschiedliche Anzahl von Zeilen zu drucken, zurück, ich werde wieder kommen ohne Fehler.

Irgendeine Idee, wie man auf die Spur das Problem, ich habe versucht zu Debuggen Sie es für Stunden, ohne Erfolg.

Ist hier der problematische code:

Linien ist definiert als char** und war malloc als:

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

void insert_line(char *s, int len){

  printf("\t\tLine Number: %d Putting a %d line into slot: %d\n",processed,len,slot);
  if(processed > numlines -1){//clean up
    free(*(lines+slot));
    *(lines + slot) = NULL;
  }
  *(lines + slot) = (char *) malloc(len * sizeof(char));
  if(*(lines + slot) == NULL) exit(EXIT_FAILURE);
  strcpy(*(lines+slot),s);
  slot = ++processed % numlines;
}
  • ist slot 0 oder 1 basiert? slot überschreitet nie zzeile-1, richtig?
  • rechts das Modul automatisch bekommen es wieder um 0
  • Dem compiler und dem debugger benutzt du? Je nachdem sind Sie in der Lage sein könnte, um etwas Hilfe bei der Fehlerbehebung Ihr problem.
  • Ich würde vorschlagen, anstelle der original-malloc Zeile mit Zeilen = (char **) calloc(zzeile, sizeof(char *)); Der Vorteil von calloc ist, dass der Speicher genullt, so dass Sie beginnen mit alle NULL-Zeiger.
InformationsquelleAutor None | 2008-10-24
  1. 7

    Ihre routine zu schreiben über den zugewiesenen line buffer.

    Die Größe der Zeile als argument übergeben (z.B. "len") wohl nicht enthalten NUL-terminator. Beim Aufruf malloc kopieren der Zeile (z.B. "s") müssen Sie reservieren ein zusätzliches byte für den string-terminator:

     *(lines + slot) = (char *) malloc((len + 1) * sizeof(char));
    • Wie hat diese Antwort akzeptiert? Einen Tag vor dieser Antwort wurde gepostet, das original-Plakat kommentiert meine Antwort, sagte: "Die null bilanziert werden von der aufrufenden Funktion."
    InformationsquelleAutor JayG
  2. 3

    Wenn Sie konsequent das problem reproduzieren mit bestimmten input-Parameter, sollten Sie die debug-so:

    • Ersten Debuggen, um die genaue frei, die das problem verursacht.
    • Dann herausfinden, wenn der Speicher, der ist etwa frei sein würde, war malloc ' ed.
    • Weiter, "debug" zu der Stelle, wo der Speicher malloc ' ed.
    • Suchen Sie in den memory viewer den zugewiesenen Speicherblock. Hinweis: start und Ende des Blocks. Es gibt wohl einen besonderen Wert genannt guard block kurz vor und kurz hinter den block.
    • Nun Schritt für Schritt durch den code, bis der Speicher frei ist würde. Irgendwann muss der code versehentlich überschreibt der guard block. Das ist die problematische Aussage.

    Beachten Sie, dass das problem könnte sehr gut sein, in einem ganz anderen Teil des Programms. Auch wenn es diesem frei, der den Fehler meldet, den code, überschreibt der guard-block kann überall sein.

    • Die null enthalten ist, den code, den Sie sehen, ist der einzige Ort, den ich betreiben auf den Zeiger zu allen. Sehr seltsam.
    • Ein anderer code sein könnte, schreiben die außerhalb seines zugewiesenen Speichers und in die guard-Werte um den block. Das ist der Grund, warum Sie brauchen, um Schritt durch den code zwischen die malloc und free. Irgendwann einige code überschreibt eine der guard-Werte.
    InformationsquelleAutor Rasmus Faber
  3. 1

    Meine erste Frage ist, wie berechnen Sie len? Ist es nur strlen oder es bietet Platz für das \0 terminator? Ich denke, man kann ein überschwingen Ihre Zuordnung in Ihrem strcpy. Schlechtes Verhalten wird dazu neigen, passieren Sie auf word-Grenzen und erscheinen zufällig. Auch, stellen Sie sicher, dass Ihre Quell-strings sind null-terminiert. Wenn du einen Fehler gemacht auf der Seite Lesen und nicht beenden Sie. Dann strcpy vielleicht zufällig überschreiben Dinge. 

      *(lines + slot) = (char *) malloc(len * sizeof(char));
  if(*(lines + slot) == NULL) exit(EXIT_FAILURE);
  strcpy(*(lines+slot),s);

    Vielleicht versuchen:

      lines[slot] = (char *) malloc((len + 1) * sizeof(char));
  if(lines[slot] == NULL) exit(EXIT_FAILURE);
  if(strlen(s) <= len){
    strcpy(lines[slot],s);
  }
  else{
    /* do something else... */
  }

    In Bezug auf die Allgemeine form, würde ich auch Sie dazu ermuntern, ein paar stilistische änderungen, um die ganze Sache ein bisschen besser lesbar, einfacher zu Folgen und resistent gegen Fehler.

    Zeiger-Arithmetik gültig ist und Spaß macht, aber ich denke, deine Absicht ist ein wenig klarer, wenn Sie das array in der form: 

    free(lines[slot]);
lines[slot] = NULL;

    statt

    free(*(lines+slot));
*(lines + slot) = NULL;

    Ich würde auch empfehlen Ihnen die Verwendung der weniger Statik. Es ist leicht genug, um durch Sie in einer Daten-Struktur, und geben Sie Sie herum in Ihren Accessoren und Mutatoren. Es wird viel mehr klar, wo die Handlung geschieht, verhindert, dass Sie Dinge wie:

    static int numlines = 0;
void insert_line(char *s, int len){
    int numlines = 5;

    wo Sie sich vorstellen können, scoping Probleme, die sind einfach miserabel zu Debuggen.

    InformationsquelleAutor Todd
  4. 0

    Tun zzeile und numlines den gleichen Wert haben?

    Macht der Aufrufer insert_line sodass Raum für die nachfolgende NUL bei der übergabe der Länge der zweite parameter?

    • ja, zzeile und numlines sind den gleichen Wert. die eigentlich Erklärung von der Linie, passiert woanders. Die null wird auf die aufrufende Funktion.
    InformationsquelleAutor Windows programmer
  5. 0

    Ich bin nicht sicher, es ist verwandt, aber diese beiden Zeilen scheint mir suspekt:

      *(lines + slot) = (char *) malloc(len * sizeof(char));
  if((lines + slot) == NULL) exit(EXIT_FAILURE);

    Weisen Sie zunächst die Rückgabe von malloc zu lines[slot] und dann überprüfen Sie (lines+slot), wenn letztere NULL war, Sie hatte dereference einen NULL-Zeiger!

    Auch wenn Linien[slot] (Ihr *(Linien+slot)) ist nicht null, Sie werden Leck-Speicher, wenn Sie übertragen das Ergebnis von malloc() zu.

    Ich gehe davon aus lines ist ein char*Zeilen[]` und der slot ist im Rahmen der erlaubten Grenze!

    InformationsquelleAutor Remo.D
  6. 0

    Stimme ich mit remo Verdacht über diese beiden Linien, aber nicht die Tangente, remo ging auf. Wir teilen sollten Kredit für das finden dieser Fehler.

    *(lines + slot) = some value
if((lines + slot) == NULL) then die
should be
if(*(lines + slot) == NULL) then die
    • Hey, Sie bearbeitet Sie Ihre source-code, während ich Bearbeite diese Antwort. In diesem Fall dein Fehler ist wahrscheinlich in einen anderen Teil des Programms, das Sie noch nicht bearbeitet noch...
    • hatte ich eigentlich schon behoben...das ist nicht, was die Ursache für meinen Fehler von malloc
    InformationsquelleAutor Windows programmer
Schreibe einen Kommentar