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 nonsense select #348

Closed
piccolbo opened this issue Mar 22, 2014 · 1 comment
Closed

segfault on nonsense select #348

piccolbo opened this issue Mar 22, 2014 · 1 comment

Comments

@piccolbo
Copy link

@piccolbo piccolbo commented Mar 22, 2014

OK, I was just messing around (trying to understand select). From fresh master branch install, R --vanilla

select(mtcars, carb + disp)

 *** caught segfault ***
address 0x504378f78, cause 'memory not mapped'

Traceback:
 1: .Call("dplyr_select_impl", PACKAGE = "dplyr", df, vars)
 2: select_impl(.data, vars)
 3: select.data.frame(mtcars, carb + disp)
 4: select(mtcars, carb + disp)

I know it shouldn't work, but I thought you'd want to catch it before a segfault. I got my lesson, I won't try it again.

@romainfrancois
Copy link
Member

@romainfrancois romainfrancois commented Apr 2, 2014

I'm no longer getting a segfault, but rather this:

> select(mtcars, carb + disp)
Erreur : invalid column index : NA for variable: NA = NA

because of:

> select_vars( names(mtcars), carb + disp )
<NA>
  NA

Not sure why select_vars does this. I'l let @hadley pick it up.

Loading

@hadley hadley closed this in 56d1ca9 Jul 28, 2014
@lock lock bot locked as resolved and limited conversation to collaborators Jun 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants