keechallenge icon indicating copy to clipboard operation
keechallenge copied to clipboard

YubiKeyWrapper.cs - Required BOTH 32bit and 64bit folders to be present.

Open goldfish1974 opened this issue 1 year ago • 0 comments

Unintuitively, both the 32bit and 64bit directories under the KeeChallenge plugin folder must exist.

I only copied the corresponding architecture folder (out of good security practices) to target desktops.

The following line (125) states the following:

if (!Directory.Exists(_32BitDir) || !Directory.Exists(_64BitDir))

If the || was changed to an && then the error condition would only fire if both 32bit and 64bit sub folders were missing.

The proposed fix would thus be:

if (!Directory.Exists(_32BitDir) && !Directory.Exists(_64BitDir))

I was forced to create an empty 32bit folder (i'm x64) to get around this problem.

goldfish1974 avatar Dec 06 '23 01:12 goldfish1974