-
Notifications
You must be signed in to change notification settings - Fork 5
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
lapack: add stricter size checks in Dlantr #69
Conversation
Is there a way to add the Draft state post hoc? These new panics will of course trigger until gonum/gonum#1207 is merged and the module info updated to pick it up. |
I had hoped that this was possible a while back, but don't think there is. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be good to have a comment in the commit message explaining this given how difficult it was to find.
We should probably start versioning this repo formally. Should we start with v0.6.0 and track the minor version of gonum/gonum? |
For certain input (documented as invalid but not checked) DLANTR in Fortran LAPACK causes out-of-bound writes. Even though this has been fixed in the Reference and will soon be in OpenBLAS, other implementations may take longer. Therefore this commit adds checks for the "invalid" matrix sizes because it's better to panic with a message rather than cause memory corruption.
Codecov Report
@@ Coverage Diff @@
## master #69 +/- ##
=========================================
- Coverage 30.23% 30.2% -0.03%
=========================================
Files 4 4
Lines 6414 6420 +6
=========================================
Hits 1939 1939
- Misses 4039 4045 +6
Partials 436 436
Continue to review full report at Codecov.
|
7084584
to
3a59b26
Compare
I've extended the commit message. Can you re-approve? |
Please take a look.