Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Files containing AAAORDID^AAA get \0-padded to 20KiB when loaded into LibreOffice only from a cryfs-monted folder #243

Closed
Gallaecio opened this issue Jan 7, 2019 · 2 comments

Comments

@Gallaecio
Copy link

Gallaecio commented Jan 7, 2019

It had been happening to me since I started working with ^-separated CSV files. When opening such a file from a cryfs-mounted folder with LibreOffice, the file size would change to 20KiB.

Steps to Reproduce the Problem

Today I’ve been able to isolate a small set of steps to reproduce the issue:

$ echo "AAAORDID^AAA" > a.csv  # Where A can be any character, as far as I could tell, the original CSV file had "RECORDID^STOREID…"
$ od -c a.csv 
0000000   A   A   A   O   R   D   I   D   ^   A   A   A  \n
0000015
$ libreoffice a.csv  # You get a dialog to import the CSV data, simply cancel
$ od -c a.csv 
0000000   A   A   A   O   R   D   I   D   ^   A   A   A  \n  \0  \0  \0
0000020  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0  \0
*
0050000

I’ve reported it here because it does not happen outside cryfs-mounted folders, only on any of my two cryfs-mounted folders. But I does not happen with applications other than LibreOffice, so if you think the issue is in their side, I’ll gladly report the issue to them instead.

Specifications

smessmer added a commit that referenced this issue Jan 13, 2019
smessmer added a commit that referenced this issue Jan 13, 2019
@smessmer
Copy link
Member

Thanks for reporting. I tracked it down and this was caused by the same race condition that caused #224 .

This is fixed.

  • for the 0.9 release series in 49a8c68 (will be in 0.9.10)
  • for the 0.10 release series in 1da3e6c (will be in 0.10-rc2)

Please build from either of these commits and let me know if it works.

@Gallaecio
Copy link
Author

Verified to be working in 0.9.10. Thank you so much!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants