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

mod 2, product of nx0 matrix and 0-dimensional vector: segfault #19378

Closed
jhpalmieri opened this issue Oct 8, 2015 · 7 comments
Closed

mod 2, product of nx0 matrix and 0-dimensional vector: segfault #19378

jhpalmieri opened this issue Oct 8, 2015 · 7 comments

Comments

@jhpalmieri
Copy link
Member

sage: matrix(GF(2), 11, 0) * vector(GF(2), 0)
------------------------------------------------------------------------
Unhandled SIGSEGV: A segmentation fault occurred in Sage.
This probably occurred because a *compiled* component of Sage has a bug
in it and is not properly wrapped with sig_on(), sig_off().
Sage will now terminate.
------------------------------------------------------------------------
Segmentation fault: 11

Component: linear algebra

Author: Travis Scrimshaw

Branch/Commit: ccfd5f8

Reviewer: John Palmieri

Issue created by migration from https://trac.sagemath.org/ticket/19378

@jhpalmieri jhpalmieri added this to the sage-6.9 milestone Oct 8, 2015
@tscrim
Copy link
Collaborator

tscrim commented Oct 10, 2015

comment:1

This has to do with the dense GF(2) matrices:

sage: matrix(GF(2), 11, 0, sparse=True) * vector(GF(2), 0)
(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0)

Here is a fix which just skips the multiplication of a vector times matrix since the result will be 0.


New commits:

ccfd5f8Don't need to multiply if the vector is 0-dimensional.

@tscrim
Copy link
Collaborator

tscrim commented Oct 10, 2015

@tscrim
Copy link
Collaborator

tscrim commented Oct 10, 2015

Author: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Oct 10, 2015

Commit: ccfd5f8

@jhpalmieri
Copy link
Member Author

comment:2

Looks good to me.

@jhpalmieri
Copy link
Member Author

Reviewer: John Palmieri

@vbraun
Copy link
Member

vbraun commented Oct 12, 2015

Changed branch from public/modules/gf2_matrix_times_0dim_vector-19378 to ccfd5f8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants