search

keepassxreboot / keepassxc

KeePassXC is a cross-platform community-driven port of the Windows application “Keepass Password Safe”.
https://keepassxc.org/
Other
21.16k stars 1.46k forks source link

CSV Parser enters infinite loop when codec is set to UTF-16 #9021

Open droidmonkey opened 1 year ago

droidmonkey commented 1 year ago

Overview

When the CSV file is NOT in UTF-16, the CSV parser will enter an infinite read loop if UTF-16 is chosen as a codec.

Steps to Reproduce

  1. Import CSV File
  2. Choose UTF-16 as the codec
  3. Infinite Loop

Expected Behavior

The parser would not enter an infinite loop

c4rlo commented 4 months ago

I cannot reproduce it with KeePassXC 2.7.8, using some test CSVs I had lying around. Does it depend on the specific CSV file?

droidmonkey commented 4 months ago

Make sure it doesn't have a BOM in the first two bytes?