{"points":200,"file":"api.php","line":3,"category":"sicherheit","evidence":"Der vollständige Bearer-Token wird in eine dauerhaft gespeicherte Logzeile übernommen.","fixOptions":[" error_log('API request ' . $path . ' with token [redigiert]');"," error_log('API request ' . $path . ' with token ' . substr($token, 0));"," error_log($token);"," error_log(base64_encode($token));"],"fixIndex":0,"fixedLine":" error_log('API request ' . $path . ' with token [redigiert]');","explanation":"Authentifizierungsdaten gehören nie in Logs. Für die Diagnose reichen Pfad, Request-ID und ein klarer Hinweis auf einen vorhandenen Token."},
{"points":200,"file":"auth.php","line":3,"category":"sicherheit","evidence":"Das Passwort wird direkt mit einem gespeicherten Wert verglichen; sichere Passwort-Hashes benötigen eine spezialisierte Prüfung.","fixOptions":[" return password_verify($passwort, $user['password_hash']);"," return md5($passwort) == $user['password'];"," return $user['password'] === $passwort;"," return strlen($passwort) > 0;"],"fixIndex":0,"fixedLine":" return password_verify($passwort, $user['password_hash']);","explanation":"password_verify prüft einen Passwort-Hash inklusive Salt und Algorithmus sicher. Klartextvergleiche und MD5 sind dafür ungeeignet."}
]
},
{
"slug":"rabatt-mit-nebenwirkung","title":"Fall #10: Rabatt mit Nebenwirkung","language":"Java","difficulty":"junior","examScope":"AP1",
"description":"Im Campus-Shop erhalten Studierende keinen Rabatt, während die Kasse bei manchen Warenkörben einen Artikel zu viel zählt.","briefing":"Vor dem Start der Prüfungswoche wurde ein neues Kassenterminal ausgerollt. Der Support meldet zwei widersprüchliche Beobachtungen: Der Rabatt erscheint bei einigen Testkonten nicht, und die Tagesstatistik weicht bei einem Warenkorb mit drei Artikeln um genau eins ab. Die Fachabteilung möchte eine kleine, nachvollziehbare Korrektur statt eines Umbaus der Kasse.","consoleOutput":"[checkout] Kundengruppe: STUDENT\n[checkout] Rabatt angewendet: false\n[stats] Warenkorb W-17: erwartet 3 Artikel, gezählt 4",
"files":[
{"filename":"Kundenprofil.java","content":"public class Kundenprofil {\n static boolean hatStudentenrabatt(String gruppe) {\n return gruppe == \"STUDENT\";\n }\n\n static int rabattProzent(String gruppe) {\n return hatStudentenrabatt(gruppe) ? 10 : 0;\n }\n}"},
{"filename":"Warenkorb.java","content":"import java.util.List;\n\npublic class Warenkorb {\n static int artikelAnzahl(List<String> artikel) {\n int anzahl = 0;\n for (int i = 0; i <= artikel.size(); i++) {\n anzahl++;\n }\n return anzahl;\n }\n}"}
],
"bugs":[
{"points":100,"file":"Kundenprofil.java","line":3,"category":"logik","evidence":"Die Gruppe kommt aus dem Profilservice und kann denselben Text in einem anderen String-Objekt enthalten.","fixOptions":[" return \"STUDENT\".equals(gruppe);"," return gruppe = \"STUDENT\";"," return gruppe.equalsIgnoreCase == \"STUDENT\";"," return gruppe != \"STUDENT\";"],"fixIndex":0,"fixedLine":" return \"STUDENT\".equals(gruppe);","explanation":"Bei Zeichenketten vergleicht == die Objektidentität, nicht den Inhalt. equals prüft den Text; die Konstante links bleibt auch bei einer fehlenden Gruppe sicher."},
{"points":100,"file":"Warenkorb.java","line":6,"category":"logik","evidence":"Die Statistik zählt pro Schleifendurchlauf. Bei drei Artikeln wird die Schleife viermal durchlaufen.","fixOptions":[" for (int i = 0; i < artikel.size(); i++) {"," for (int i = 1; i <= artikel.size(); i++) {"," for (int i = 0; i <= artikel.size() - 1; i += 2) {"," for (int i = artikel.size(); i >= 0; i--) {"],"fixIndex":0,"fixedLine":" for (int i = 0; i < artikel.size(); i++) {","explanation":"Die Anzahl der Durchläufe reicht von Index 0 bis vor die Größe der Liste. Mit < artikel.size() wird jeder Artikel genau einmal gezählt."}
]
},
{
"slug":"freigabe-vor-pruefung","title":"Fall #11: Freigabe vor Prüfung","language":"Java","difficulty":"senior","examScope":"AP2_FIAE",
"description":"Ein bereits abgelaufener Release wird im Deployment-Portal als freigegeben angezeigt, und ein API-Schlüssel taucht im Diagnoseprotokoll auf.","briefing":"Das Release-Team stoppte ein Hotfix-Deployment wenige Minuten vor dem Rollout. Die Oberfläche zeigte trotz abgelaufener Freigabe ein grünes Licht. Parallel hat das Security-Team in einem zentralen Log einen produktiven API-Schlüssel entdeckt. Untersuche die Entscheidungslogik und die Diagnoseausgabe; beide Fehler liegen in verschiedenen Komponenten und dürfen nicht mit einem bloßen Workaround kaschiert werden.","consoleOutput":"[release] Status: GEPRUEFT\n[release] Freigabe gültig bis: 2026-07-12T18:00:00Z\n[release] Bewertung um 2026-07-13T08:14:00Z: freigegeben\n[audit] deployment payments-api mit key sk_live_...",
{"points":150,"file":"DeploymentFreigabe.java","line":6,"category":"logik","evidence":"Der Zeitvergleich ist korrekt; der Status stammt jedoch aus einer Datenbankabfrage und wird als String geliefert.","fixOptions":[" && \"GEPRUEFT\".equals(release.status());"," && release.status() = \"GEPRUEFT\";"," && release.status().equals = \"GEPRUEFT\";"," && release.status() != \"GEPRUEFT\";"],"fixIndex":0,"fixedLine":" && \"GEPRUEFT\".equals(release.status());","explanation":"Der Freigabestatus muss inhaltlich verglichen werden. Mit == kann derselbe Text je nach Herkunft fälschlich als verschieden behandelt werden."},
{"points":150,"file":"AuditLog.java","line":3,"category":"sicherheit","evidence":"Der Schlüssel ist ein Credential. Zentrale Logs sind für Betriebsteams einsehbar und oft länger gespeichert als die eigentliche Sitzung.","fixOptions":[" System.out.println(\"deployment \" + releaseName + \" mit key [redigiert]\");"," System.out.println(\"deployment \" + releaseName + \" mit key \" + apiKey.substring(0));"," System.out.println(apiKey);"," System.out.println(\"deployment \" + releaseName + \" mit key \" + java.util.Base64.getEncoder().encodeToString(apiKey.getBytes()));"],"fixIndex":0,"fixedLine":" System.out.println(\"deployment \" + releaseName + \" mit key [redigiert]\");","explanation":"API-Schlüssel dürfen nicht protokolliert werden – auch nicht kodiert oder vollständig abgeschnitten. Für die Diagnose reichen Release-Name, Zeit und eine Request-ID."}
]
},
{
"slug":"bericht-mit-fremdzugriff","title":"Fall #12: Bericht mit Fremdzugriff","language":"PHP","difficulty":"principal","examScope":"AP2_FIAE",
"description":"Ein Mandantenbericht lässt sich über einen manipulierten Dateinamen außerhalb des Archivbereichs aufrufen; außerdem prüft die Berechtigung den Mandanten nicht.","briefing":"Ein externer Penetrationstest hat zwei Befunde kombiniert: Eine angemeldete Person kann einen ungewöhnlichen Berichtsnamen an den Download-Endpunkt senden und die Zugriffsprüfung bestätigt nur eine allgemeine Rolle. Im Incident-Log steht eine Anfrage, die auf einen fremden Mandanten zeigt. Die Lösung muss Eingaben begrenzen und den Zugriff an den tatsächlich angeforderten Kunden binden, ohne den normalen Monatsreport zu beschädigen.","consoleOutput":"GET /reports/download?bericht=../../kunden/K-204/umsatz\n[report] Datei angefordert: ../../kunden/K-204/umsatz\n[auth] user U-88, role REPORT_VIEWER, angeforderter Kunde K-204\nSecurity finding: CWE-22 Path Traversal\nSecurity finding: CWE-639 Authorization Bypass Through User-Controlled Key",
"files":[
{"filename":"ReportController.php","content":"<?php\nfunction downloadReport(): string {\n $name = $_GET['bericht'] ?? '';\n $file = REPORT_DIR . '/' . $name . '.pdf';\n if (!is_file($file)) {\n throw new RuntimeException('Bericht nicht gefunden');\n }\n return file_get_contents($file);\n}\n?>"},
{"points":200,"file":"ReportController.php","line":3,"category":"sicherheit","evidence":"Der Dateiname kommt direkt aus dem Query-Parameter. Die anschließende Dateiprüfung verhindert keinen Pfadwechsel aus REPORT_DIR heraus.","fixOptions":[" $name = $_GET['bericht'] ?? ''; if (!preg_match('/^[A-Z0-9_-]+$/', $name)) { throw new InvalidArgumentException('Ungültiger Bericht'); }"," $name = str_replace('../', '', $_GET['bericht'] ?? '');"," $name = base64_encode($_GET['bericht'] ?? '');"," $name = $_GET['bericht'] ?? ''; chmod(REPORT_DIR, 0777);"],"fixIndex":0,"fixedLine":" $name = $_GET['bericht'] ?? ''; if (!preg_match('/^[A-Z0-9_-]+$/', $name)) { throw new InvalidArgumentException('Ungültiger Bericht'); }","explanation":"Eine Positivliste erlaubt nur die erwarteten Berichtsschlüssel. Das bloße Entfernen einzelner Zeichenfolgen lässt Varianten von Pfadmanipulationen offen."},
{"points":200,"file":"ReportAccess.php","line":3,"category":"sicherheit","evidence":"Die Rolle erlaubt das Lesen von Berichten, aber sie beweist nicht, dass der angeforderte Kunde zum Benutzer gehört.","fixOptions":[" return $user['role'] === 'REPORT_VIEWER' && in_array($kundeId, $user['kundenIds'], true);"," return $user['role'] === 'REPORT_VIEWER' || $kundeId !== '';"," return $user['role'] == 'REPORT_VIEWER' && $kundeId == $kundeId;"," return strlen($kundeId) > 0;"],"fixIndex":0,"fixedLine":" return $user['role'] === 'REPORT_VIEWER' && in_array($kundeId, $user['kundenIds'], true);","explanation":"Autorisierung braucht sowohl die passende Rolle als auch eine Mandantenbindung. Der strikte Vergleich in in_array verhindert dabei ungewollte Typumwandlungen."}
<!doctype html><htmllang="de"data-theme="dark"><head><metacharset="utf-8"><metaname="viewport"content="width=device-width,initial-scale=1"><title>BugHunter · Mindflux</title><linkrel="stylesheet"href="/css/theme.css"><linkrel="stylesheet"href="/css/bughunter.css?v=2"><scriptsrc="/js/theme.js"></script></head><body><mainclass="bh-app"><headerclass="bh-top"><ahref="/dashboard"class="bh-back">← Dashboard</a><divclass="bh-brand"><b>BUGHUNTER</b><span>DEBUG-FALLAKTEN</span></div><spanclass="mf-a11y-host"></span></header><sectionid="bhView"class="bh-view"><divclass="bh-loading">FALLAKTEN WERDEN GEÖFFNET …</div></section></main><scriptsrc="/js/bughunter.js?v=2"></script></body></html>
<!doctype html><htmllang="de"data-theme="dark"><head><metacharset="utf-8"><metaname="viewport"content="width=device-width,initial-scale=1"><title>BugHunter · Mindflux</title><linkrel="stylesheet"href="/css/theme.css"><linkrel="stylesheet"href="/css/bughunter.css?v=2"><scriptsrc="/js/theme.js"></script></head><body><mainclass="bh-app"><headerclass="bh-top"><ahref="/dashboard"class="bh-back">← Dashboard</a><divclass="bh-brand"><b>BUGHUNTER</b><span>DEBUG-FALLAKTEN</span></div><spanclass="mf-a11y-host"></span></header><sectionid="bhView"class="bh-view"><divclass="bh-loading">FALLAKTEN WERDEN GEÖFFNET …</div></section></main><scriptsrc="/js/bughunter.js?v=3"></script></body></html>
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
AliceDarkest