anakinraw / falsecrypt Goto Github PK

Work looks done for me, but can only create pull request for master, I need another branch for java implementation. https://github.com/jusito/FalseCrypt/tree/master-java

Sollte funktionieren, Maven goal "package" -> target/FalseCrypt-0.0.1-SNAPSHOT-jar-with-dependencies.jar

Möglicherweise ungewollte Bugs:
1.
https://github.com/AnakinSklavenwalker/FalseCrypt/blob/master/src/FalseCrypt.Crypto/CryptoWrapper.cs
Zeile 45 ist "password" hardkodiert obwohl password übergeben wird?
Wurde in Java korrigiert entfernt bzw die variable genutzt. (sofern als Passwort nur "password" genutzt wird fällt das nicht auf)

View:
Bug 1) etwas abgeändert, es werden aber immernoch für alle Dateien unterhalb des Verzeichnisses derselbe Key genutzt
WeakCryptoConfig:
Bug 2) identisch
WeakKeyGenerator:
Bug 3) identisch
Bug 4) Klassenvariable in Java funktioniert so nicht, diese wird nur einmal beim laden der Klasse erstellt, wird jetzt bei jedem Aufruf in der Methode neu erstellt. (wäre für mich in C# unerwartetes Verhalten gewesen)
Bug 5) identisch
WeakPasswordDerivation:
Bug 6) In Java wählt man die Provider nicht selbst, damit ist es nicht mehr plattformunabhängig (https://docs.oracle.com/javase/8/docs/technotes/guides/security/SunProviders.html)
Bug 7) Provider hat kein close / dispose -> funktioniert in Java nicht
Bug 8) identisch
Bug 9) identisch
Bug 10) kein closable / dispose aber PBEKeySpec erlaubt "clearPassword" und secretKey "destroy"
Bug 11) sollte identisch sein
WeakSymmetricEncryption
Bug 12) Siehe 6, oder ist der Algorithmus (DES) gemeint? Provider wäre z.B. Sun JCE
Bug 13) identisch
Bug 13.1) Aber das Padding wird nicht kritisiert? => Padding Oracle, wobei PKCS5 in java genutzt werden musste, sollte hier identisch zu PKCS7 sein und ermöglicht auch padding oracle.
Bug 14) Cipher muss nicht disposed / closed / destroyed werden -> geht hier nicht
Bug 15/16) geht nicht, beim einkommentieren der Zeile, welche korrekt ist, wird zur Laufzeit eine Exception geworfen, dass der IV in ECB nicht genutzt werden kann
Bug 17) SecretKeySpec ermöglicht ein destroy, keine Ahnung ob das nötig ist
Bug 18) identisch
Bug 19) identisch
Bug 20) siehe 6 und 12
Bug 21) identisch
Bug 21.1) wie 13.1?
Bug 21.2) CipherInputStream nicht closed

Als nächstes schaue ich mir mal gradle und Android an.