C Passwortabfrage (aus Youtube-Video)

[Bartosz B. (bartosz)]

Hallo zusammen,

ich habe 1:1 dieses Youtube-Video nachgeschrieben und dachte erst, dass 
es eine Sache von ner Viertelstunde sei. Doch mein System spielt mir 
einen Streich...

https://www.youtube.com/watch?v=EEmAAgOjDFg

Bei mir liest getch() nach dem eigentlichen Buchstaben zusätzlich '\0' 
ein, sodass die while-Schleife ein 2. Mal durchläuft, sodass ein 2.Mal 
ein * geprintet wird und sodass der String nicht länger wird!

Was läuft hier schief?

PS: ich habe seine Steuerzeichen durch '\r' ersetzt, finde ich schöner.

bool Passwortabfrage(string richtigesPasswort)
{
  char einZeichen; //letztes, eingegebenes Zeichen
  string ganzesPasswort;
  einZeichen = getch();

  while (einZeichen != '\r')  //solange nicht ENTER...
  {
    if (einZeichen != '\b') //wenn nicht BACKSPACE...
    {
      ganzesPasswort = ganzesPasswort + einZeichen;
      putch('*');
    }
    else if (einZeichen == '\b' && ganzesPasswort.length() > 0)
    {
      ganzesPasswort.resize(ganzesPasswort.length() - 1);
      putch('\b');  //Cursor nach links
      putch(' ');  //überschreiben
      putch('\b'); //wieder auf Leerzeichen draufspringen
    }
    einZeichen = getch();
  }

  if (ganzesPasswort == richtigesPasswort)
  {
    return true;
  }
  else
  {
    return false;
  }
}


*In der main*

for (int i = 0; i <= 2; i++)
  {
    printf("Passwort: ");
    if (Passwortabfrage("hallo Welt"))
    {
      printf("\nOK\n");
      break;      //aus for-Schleife raus
    }
    else
    {
      Beep(500, 500);
      Beep(300, 500);
      printf("Passwort ist falsch!\n");
    }
  }

[foobar (Gast)]

> Doch mein System spielt mir einen Streich...

Du solltest das System nennen, denn üblich ist dieses Verhalten nicht.

[Bartosz B. (bartosz)]

foobar schrieb:
>> Doch mein System spielt mir einen Streich...
>
> Du solltest das System nennen, denn üblich ist dieses Verhalten nicht.

Windows 8
Entwicklungsumgebung ist Visual Studio Community

[foobar (Gast)]

Könnte dieser Bug im Windows sein:
  https://stackoverflow.com/questions/51792349/getch-returns-2-characters-when-i-type-one

[foobar (Gast)]

s/Windows/C-Runtime/

[Bartosz B. (bartosz)]

Danke dir!
mit _getwch() funktioniert alles. Hab's gerade mit richtigem und 
falschem Passwort getestet.

[DPA (Gast)]

Bartosz B. schrieb:
> mit _getwch() funktioniert alles.

_getwch ist eine Windowsspezifische non-standard funktion, selbes gillt 
für _getch. Und getch ist normalerweise eine curses funktion, curses ist 
eine Library für Konsolenprogramme. Die Standardfunktion im c standard 
zum Lesen eines Zeichens ist getchar. Alternativ kann man auch getc und 
fread nutzen. Auf POSIX compatiblen systemen ist ausserdem die 
Verwendung von "read" möglich. Wenn man mit diesen ein Passwort einlesen 
will, sollte man aber erst das terminal echo ausschalten. Auf posix 
systemen macht man das über das ECHO flag mit tcsetattr (termios). 
Ausserdem sollte aus Sicherheitsgründen wenn möglich nicht von stdin, 
sondern vom controling terminal gelesen werden. Ausserdem kann es je 
nach threat model sinn machen, secure memory zu verwenden (speicher, der 
im idealfall nicht geswappt werden kann und beim free genullt wird. Eine 
implementation wäre gcry_malloc_secure.). Windows kocht überall halt wie 
immer sein eigenes Süppchen, mit dem winapi mist. Ich empfehle ein 
richtiges OS zu verwenden, nicht so ein proprietäres frickelsystem wie 
Windows.

Besser währe eventuell noch, gleich die vorgesehenen Funktionen dafür zu 
nehmen, statt das selber zu machen. Die POSIX Standardfunktion zum 
einlesen von Passwörtern ist getpass. Unter BSD gibt es das verbesserte 
readpassphrase.

Wenn möglich ist die Verwendung von pam zur Authentifizierung zu 
bevorzugen. http://man7.org/linux/man-pages/man3/pam.3.html

Jenachdem kann es gar sinvoll sein, Programme wie su zu verwenden.

Bei Terminals ist es ausserdem unüblich, die Anzahl eingegebener Zeichen 
mit Sternchen anzuzeigen. Das ist ein Sicherheitsaspeckt, wenn man die 
Passwortlänge kennt, ist es schon um einiges leichter, das Passwort zu 
erraten.

[ohne Account (Gast)]

> _getwch ist eine Windowsspezifische non-standard funktion, selbes gillt
> für _getch. Und getch ist normalerweise eine curses funktion, curses ist
> eine Library für Konsolenprogramme. Die Standardfunktion im c standard
> zum Lesen eines Zeichens ist getchar.
Er benutzt die C99 oder C11 Variante und der Rest ist dann auch nicht 
weiter erwähnenswert.