garlicpasta/rechnersicherheitss2017

rechnersicherheitss2017's Issues

ushout issues

• assignment 2
  • keine warning flags in der Makefile? -Wall würde man schon erwarten
  • kein check auf root?
  • memory disclosure in log file mangels \0 in buf in :77.
  • das mit den Buffer-Größen ist stellenweise verwirrend.
• assignment 4
  • kompiliert nicht: -std=c99 fehlt
  • passwörter im klartext zu speichern ist jetzt nicht ganz "as secure as possible"
  • in :57: user_data[i++] = token; buffer overflow, falls pw mehrere ":" enthält
  • Vorsicht bei malloc(a*b), besser: calloc(a,b) (wegen int-overflows)
  • in :134: buffer ist eigentlich nur 130 Zeichen lang (siehe :117)
  • in :193: wenn string ohne \0 gesendet wird (z.B. BUFFER_SIZE lang) dann schreibt das eine 0 irgendwo hin -> buffer overflow
  • analog :205
  • ist der zugriff auf clients[i] irgendwie synchronisiert?
  • werden clients auch wieder entfernt?
  • nur 1 user unterstützt? (siehe :71)
  • zu :76: schon mal geschaut was strcmp zurückgibt, wenn gleich? (->man page) Das würde erklären warum es immer klappt.
  • FYI: clients_len ist immer 20, clients[i] dann auf 0 initialisiert -> deswegen so viele duplikate in stdout
  • client_message wird nie gezeroed, das strlen führt also immer dazu dass die vorherige Nachricht mitgesendet wird (tip: probiert mal ctrl-d in nc)