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

[ONNX Runtime] Fix OrtNDArray double free issue #1861

Merged
merged 1 commit into from
Aug 3, 2022
Merged

[ONNX Runtime] Fix OrtNDArray double free issue #1861

merged 1 commit into from
Aug 3, 2022

Conversation

xyang16
Copy link
Contributor

@xyang16 xyang16 commented Aug 3, 2022

Description

Brief description of what this PR is about

Fix crash caused by double close OrtNDArray

  • If this change is a backward incompatible change, why must this change be made?
  • Interesting edge cases to note here

@xyang16 xyang16 changed the title Arm support [ONNX Runtime] Fix OrtNDArray double close issue Aug 3, 2022
@xyang16 xyang16 merged commit 9c8761e into deepjavalibrary:master Aug 3, 2022
@xyang16 xyang16 self-assigned this Aug 3, 2022
@xyang16 xyang16 changed the title [ONNX Runtime] Fix OrtNDArray double close issue [ONNX Runtime] Fix OrtNDArray double free issue Aug 4, 2022
patins1 pushed a commit to patins1/djl that referenced this pull request Aug 26, 2022
@xyang16 @patins1
[ONNX Runtime] Fix OrtNDArray double close issue (deepjavalibrary#1861)
abe9b0f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frankfliu frankfliu frankfliu approved these changes

@zachgk zachgk Awaiting requested review from zachgk zachgk is a code owner

Assignees

@xyang16 xyang16

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xyang16 @frankfliu