4. PrettyOS Fehler-/Testthread

PrettyOS Fehler-/Testthread

  • E
    Mod

    Wer schreibt eigentlich an der Shell? Würde mich da gern einbringen und unterstützend wirken.

    Ich schreibe an der shell. Vielleicht findet jemand heraus, wie man die Abfrage der shell so umbauen kann, dass sowohl

    "hello", "hello.elf", "hello .elf", "hello  .elf", "hello   .elf"

    (1,4,5 geht, wenn ich das richtig sehe; 2 u. 3 werden als ".ELF" gewertet) als

    "HELLO   ELF"

    interpretiert wird. Da habe ich mich irgendwie verhaspelt, das ging sogar schon.

    Ebenso was externe Programme betrifft.

    Das machen wohl einige momentan. Gute Ideen werden hier gerne gesehen. Man muss ja nur hello.c austauschen gegen eigenen Code. Allerdings sind die userlib.h/c und die syscalls noch ziemlich rudimentär. Anregungen werden gerne entgegen genommen.

    0
  • X

    Erhard Henkes schrieb:

    Vielleicht findet jemand heraus, wie man die Abfrage der shell so umbauen kann, dass sowohl

    "hello", "hello.elf", "hello .elf", "hello  .elf", "hello   .elf"

    (1,4,5 geht, wenn ich das richtig sehe; 2 u. 3 werden als ".ELF" gewertet) als

    "HELLO   ELF"

    interpretiert wird.

    Mit einem svn diff von

    Index: user/user_program_c/program.c                               
===================================================================
--- user/user_program_c/program.c       (Revision 91)              
+++ user/user_program_c/program.c       (Arbeitskopie)             
@@ -149,7 +149,7 @@                                                
                   }                                               
               }                                                   

-              for(i=posPoint+1;i<12;i++)                          
+              for(i=posPoint+1;i<posPoint+4;i++)                  
               {                                                   
                   ext[i-posPoint-1]=entry[i];                     
               }

    funktioniert es, also eine Ersetzung in Zeile 152 der benannten Datei.

    Ohne diese Änderung gibt es einen Pufferüberlauf auf dem Stack, somit wird "name" überschrieben und am Ende kommt ".ELF" raus ("name" beginnt mit '\0').

    0
  • E
    Mod

    Merkwürdig, ich wusste doch, dass das schon geklappt hat. Danke nochmals. 🙂

    0
  • X

    Du schreibst auch vieles, was du im IRC sagst, nochmal ins Forum. 😉

    0
  • E
    Mod

    Wir loggen das IRC nicht zentral mit, daher ist das "Forum" ein wichtiges Kommunikationsmedium, in dem der Fortschritt und die Wirren von PrettyOS festgehalten werden sollten. Daher die Bitte, alle wichtigen Beiträge hier ebenfalls einzupflegen. 🙂

    0
  • C

    Rev. 103:

    PrettyOS erkennt nur großgeschriebene Dateien, ein Fehler, den ich sehr spät bemerkt habe..-.- Ist etwas ungünstig, da ich aus Gewohnheit immer kleine Buchstaben verwende.

    0
  • M

    Das kann ich nicht bestätigen, Cuervo...

    0
  • X

    Ich auch nicht...

    Aber ich habe ein anderes Problem: Wenn ich hello.elf starte und weiter Backspace mache, als eigentlich möglich ist (und danach dann normal weiterspiele), bekomme ich beim Beenden einen Pagefault:

    page not present at user-mode (errcode = 0x00000004)
CR2 = 0x08080804
CS:EIP = 0x001B:0x0140050D
SS:ESP = 0x0023:0x08080804
EFLAGS = 0x00000212
EAX = 0x00000000
EBX = 0x08080808
ECX = 0x08080808
EDX = 0x00000000
ESI = 0x08080808
EDI = 0x08081B08
EBP = 0x08080808
DS = 0x0023
ES = 0x0023
FS = 0x0023
GS = 0x0023

    Ohne in den Code gesehen zu haben würde ich raten, dass da ein Stackoverflow geschieht: Vermutlich wird die Eingabe des Benutzers in einem Array auf dem Stack gespeichert. Wenn man zu oft Backspace drückt, dann wird der ganze Stack mit \b (also 0x08) überschrieben und dann werden am Ende des Programms die gespeicherten Register vom Stack geholt (so entstehen 0x08080808 in EBX, ECX, ESI, (EDI) und EBP). Dahinter steht dann wohl noch eine C-Funktion, die am Ende den Stackframe mit "mov esp,ebp" und "pop ebp" aufzulösen versucht. Erstere Funktion führt dazu, dass 0x08080808 in ESP geladen wird und die zweite zieht zuerst 4 von ESP ab (sodass 0x08080804 drinsteht) und versucht dann, einen Wert von dieser Adresse zu laden. Diese ist offensichtlich nicht gemappt und das führt dann zum PF an dieser Adresse.

    q.e.d. 😃

    EDIT: PS, bitte mal die Vordergrundfarbe bei Panics ändern. Dieses Dunkelrot ist echt nicht zu erkennen, da macht man sich die Augen kaputt. Hellrot wäre besser. 😉

    0
  • C

    komisch, also wenn ich z.B. hello eingebe, wird daraus HELLO .ELF und hello.elf wird nur gefunden, wenn es groß geschrieben ist (auf der Diskette)...

    0
  • X

    Cuervo schrieb:

    komisch, also wenn ich z.B. hello eingebe, wird daraus HELLO .ELF und hello.elf wird nur gefunden, wenn es groß geschrieben ist (auf der Diskette)...

    Natürlich wird daraus HELLO.ELF, auf FAT wird Groß- und Kleinschreibung nicht unterschieden und es ist vorgeschrieben, dass Dateinamen als Großbuchstaben (zumindest ohne VFAT) gespeichert werden.

    0
  • C

    mag sein, PrettyOS zeigt jedenfalls hello.elf unter fdir an statt HELLO.ELF und findet es nicht, vermutlich weil die Eingabe in Großbuchstaben umgewandelt wird.

    0
  • X

    Wenn du in den Quellcode der Shell sehen würdest, könntest du sehen, dass das tatsächlich so ist. 😉

    0
  • C

    Dann hatte ich ja Recht^^

    else
{
puts("file is being searched.\n");
settextcolor(2,0);
toupper(entry);

    Also ist das ein Fehler, weil die Datei ja hello.elf und nicht HELLO.ELF heißt.

    0
  • E
    Mod

    vermutlich weil die Eingabe in Großbuchstaben umgewandelt wird

    Wie gesehen, machen wir das FAT12-gerecht. Kleinschreibung ist nicht vorgesehen für Filename und Extension. Wenn also von anderen Sytemnen auf die Diskette geschrieben wird, bitte nur Großschreibung verwenden.

    Wenn es dich momentan stört, nimmst Du das toupper(entry); eben raus.

    Damit haben wir den üblichen Konflikt zwischen Windows Client und Linux Server, wenn ich das richtig sehe.

    Wie wird das allgemein im OSDEV-Bereich gehandhabt?

    0
  • X

    Cuervo schrieb:

    Also ist das ein Fehler, weil die Datei ja hello.elf und nicht HELLO.ELF heißt.

    Bei FAT ist es (ohne VFAT) halt einfach vorgeschrieben, dass der Dateiname großgeschrieben wird. Deshalb gibt es keinen Unterschied zwischen "hello.elf" und "HELLO.ELF".

    EDIT: vgl. http://en.wikipedia.org/wiki/8.3_filename#Overview: "File and directory names are uppercase, although systems that use the 8.3 standard are usually case-insensitive."
    oder auch http://en.wikipedia.org/wiki/File_Allocation_Table#Directory_table: "Legal characters for DOS file names include the following: Upper case letters A–Z [...]" -- "This excludes the following ASCII characters: [...] Lower case letters a–z; Stored as A–Z. [...]"

    0
  • E
    Mod

    Weiß eigentlich jemand, warum man das damals so entschieden hat? Linux ist ja im Gegensatz dazu case-sensitive.

    0
  • B

    Der Zug bewegt sich bei mir immer unter Bochs nur weiter, wenn ich eine Taste drücke (oder ist das Absicht?).

    0
  • M

    Das ist überall so und liegt nicht an Bochs...

    0
  • E
    Mod

    Der Zug bewegt sich bei mir immer unter Bochs nur weiter, wenn ich eine Taste drücke (oder ist das Absicht?).

    Ja, das hat sich so ergeben, weil wir den inneren Loop nun nicht in der gesamten Shell, sondern nur in getch drehen (siehe Definition von getch in user-Lib). Wir fangen die vom Kernel zurück gegebene 0 dort ab.

    0
  • A

    Unter VirtualBox v3.1.4 r57640 stürtz PrettyOS Rev. 120 ab. Scheinbar geschieht dies, nachdem das Datum ausgegeben wird. Falls jemand interesse an der Log-Datei von VBox hat, ist diese hier zu finden: http://anubis2k5.bplaced.net/VBox.log

    Unter Qemu läuft das OS normal.

    0
Anmelden zum Antworten