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

Segfault on call to fread with fill = TRUE #4774

Closed
AndeolEvain opened this issue Oct 21, 2020 · 2 comments · Fixed by #5046
Closed

Segfault on call to fread with fill = TRUE #4774

AndeolEvain opened this issue Oct 21, 2020 · 2 comments · Fixed by #5046

Comments

@AndeolEvain
Copy link

I have a file that systematically makes R process crash when calling the following code :

dn1 <- data.table::fread("minimalEx.csv",
        sep = ",",
        quote = "\"",
        header = TRUE,
        fill = TRUE,
        stringsAsFactors = TRUE,
        integer64 = "numeric"
)

When trying to make a minimal example, I found that :
_ There seem to be a minimal number of lines (around 100+) for this crash to happen. Below this threshold, I simply get an error.
_ It is linked to non-escaped quotes, here in that last line of the file
_ It only happens with argument fill = TRUE

Here is the file reduced to a minimal example :
minimalEx.zip

This seems somwhat linked to the bug reported in # #3509, but the conditions of reproduction are a bit different.

@jangorecki
Copy link
Member

Thank you for reporting. For completeness would be good to have your session info as well.

@AndeolEvain
Copy link
Author

I was on R 3.6.3 (on windows 10). Then reproduced the same issue on R 3.5.3. I updated the data.table package when trying to find the minimal case (not sure what was the version before), so I now have the version 1.12.8.
My collegues also reproduced the issue with R3.6.0, and on Mac with R3.5.3, so it doesn't seem too system-dependant.

@mattdowle mattdowle added this to the 1.14.1 milestone Jun 21, 2021
@jangorecki jangorecki modified the milestones: 1.14.9, 1.15.0 Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants