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.

Sign up for GitHub

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

Jump to bottom

Python engine variables double free crash #2950

Closed
StasJ opened this issue Dec 30, 2021 · 1 comment · Fixed by #2955
Closed

Python engine variables double free crash #2950

StasJ opened this issue Dec 30, 2021 · 1 comment · Fixed by #2955
Assignees
@clyne
Labels
Bug High
Milestone
3_6_0 Release

Comments

@StasJ
Copy link
Collaborator

StasJ commented Dec 30, 2021

  1. Open a dataset
  2. Create a valid python variable and save it
  3. Create a new session
  4. Open a dataset

Vapor will crash. Just replacing the dataset in the above steps (as in import -> replace existing) should cause a crash too.
StasJ added a commit that referenced this issue Dec 30, 2021
@StasJ
Fix #2950
8df0728
@StasJ
Copy link
Collaborator Author

StasJ commented Dec 30, 2021 •
edited
Loading

There is a stopgap fix in the branch issue2950.

@sgpearse sgpearse added this to the 3_6_0 Release milestone Jan 4, 2022
clyne added a commit that referenced this issue Jan 5, 2022
@clyne
Fix #2950
3ee4e0e
clyne added a commit that referenced this issue Jan 5, 2022
@clyne
Fix #2950
ff1e9ce
@clyne clyne mentioned this issue Jan 5, 2022
Merged
clyne added a commit that referenced this issue Jan 6, 2022
@clyne @StasJ
Fixes #2950 - Python engine variables double free crash (#2955)
0e4813a 
* Fix #2950

* fix

* Fix #2950

* clang-format pre-push hook

* Fix #2950

* Documentation for CalcEngineMgr::Clean()

Co-authored-by: StasJ <stas708@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clyne clyne

Labels
Bug High
Projects
None yet
Milestone
3_6_0 Release
Development

Successfully merging a pull request may close this issue.

Fixes #2950 - Python engine variables double free crash NCAR/VAPOR
3 participants
@StasJ @clyne @sgpearse