NullPointerException bei Zugriff auf eingeschränkte Verzeichnisse
Closed this issue · 6 comments
Beim lese-Versuch tritt bei geschützten Verzeichnissen unter Windows ("System Volume Information" und andere) eine NPE im DuplicateLengthFinder.DuplicateLengthRunner in Zeile 85 auf.
Hotfix in Vorbereitung
Als nächstes integriere ich diese Info in den Callback, damit ist der Bug dann gefixt.
Bug noch reproduzierbar: Zugriff auf Unterverzeichnis des Windows Papierkorbs.
Ich würde als erstes die checkFolder Methode optimieren. Aus meiner Sicht könnten mehrere Bedingungen für einen Folder vorliegen. Zum Beispiel könnte ein leerer nicht lesbarer Folder vorliegen. Dann würde ich beim Ermitteln der Liste schon eine PermissionException erwarten, noch bevor canRead ermittelt werden kann. Bei der ersten Fehlerbedingung würde ich mit Exception aussteigen - wozu eine Message als Rückgabewert? Für den Nutzer? Genügt da nicht ein logging vor dem Werfen einer Exception?
Ein Stacktrace wäre gut gewesen, aktuell ist die Zeile 85 ein Kommentar.
Die else if sind im master auch nicht enthalten. In welchem Branche würde denn gefixt?
Sorry, ich wollte gerade den Review durchführen aber ich finde keinen Branch oder Pull Request welcher die Deltas enthält. #2 ist nur ein Issue aber ohne zugehörigen Branch. Kannst Du mir einen Branch oder eine Commit ID geben welche die Deltas enthält damit ich den Review gegen den master durchführen kann?