From fe06e9f45646893dc88ebe9e657e2e991f6f5fbb Mon Sep 17 00:00:00 2001 From: Jakob Unterwurzacher Date: Mon, 8 Apr 2019 20:18:45 +0200 Subject: readpassword: delete CheckTrailingGarbage CheckTrailingGarbage was called even when "-passfile" was used, which is stupid, and causes false positives: https://github.com/rfjakob/gocryptfs/issues/391 (false error "Received trailing garbage after the password" when using -passfile in .bash_profile) Instead of trying to improve the logic to handle that case and make everything even more complicated, delete the function. It is unclear if actually helps in some cases, and it definitely harms as shown by the above bug report. --- init_dir.go | 1 - 1 file changed, 1 deletion(-) (limited to 'init_dir.go') diff --git a/init_dir.go b/init_dir.go index c3aa4b5..a9c66e3 100644 --- a/init_dir.go +++ b/init_dir.go @@ -81,7 +81,6 @@ func initDir(args *argContainer) { } else { // Normal password entry password = readpassword.Twice([]string(args.extpass), args.passfile) - readpassword.CheckTrailingGarbage() } creator := tlog.ProgramName + " " + GitVersion err = configfile.Create(args.config, password, args.plaintextnames, -- cgit v1.2.3