Skip to content
This repository has been archived by the owner on Nov 13, 2023. It is now read-only.

Reported segfault in SmartDashboard.putStringArray #649

Closed
virtuald opened this issue Feb 14, 2020 · 3 comments
Closed

Reported segfault in SmartDashboard.putStringArray #649

virtuald opened this issue Feb 14, 2020 · 3 comments
Labels
Projects

Comments

@virtuald
Copy link
Member

/cc @celttechie

Windows 10, Python 3.8.1 64-bit. Might be robotpy/robotpy-wpiutil#7?

@auscompgeek
Copy link
Member

Quite possibly if the strings are computed. I imagine constants would work by chance.

@auscompgeek
Copy link
Member

Actually, hmm. ntcore stores a std::vector<std::string> and references that. vector and string own their contents, right?

@virtuald
Copy link
Member Author

I looked at this a bit deeper, and I haven't been able to reproduce. I think if this were to happen, it would be that somehow an invalid py::str was returned from a function somewhere and added to an array that was used. Lots of pybind11 churn this summer, including a bunch of string stuff, so it's likely this has been addressed.

2021 Season automation moved this from To do to Done Feb 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
No open projects
2021 Season
  
Done
Development

No branches or pull requests

2 participants