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

fix SBox __init__ #9366

Closed
sagetrac-ylchapuy mannequin opened this issue Jun 28, 2010 · 7 comments
Closed

fix SBox __init__ #9366

sagetrac-ylchapuy mannequin opened this issue Jun 28, 2010 · 7 comments
Assignees
@sagetrac-mvngu
Labels
c: cryptography p: major / 3 t: bug
Milestone
sage-4.5.2

Comments

@sagetrac-ylchapuy
Copy link
Mannequin

sagetrac-ylchapuy mannequin commented Jun 28, 2010

the input size was badly checked.

CC: @malb

Component: cryptography

Author: Yann Laigle-Chapuy

Reviewer: Martin Albrecht

Merged: sage-4.5.2.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/9366
@sagetrac-ylchapuy sagetrac-ylchapuy mannequin added this to the sage-4.5.2 milestone Jun 28, 2010
@sagetrac-ylchapuy
Copy link
Mannequin Author

sagetrac-ylchapuy mannequin commented Jun 28, 2010

requires #9318

@sagetrac-ylchapuy

This comment has been minimized.

Sign in to view
@sagetrac-ylchapuy
Copy link
Mannequin Author

sagetrac-ylchapuy mannequin commented Jun 28, 2010

comment:1

Attachment: trac9366_fix_SBox_init.patch.gz

@sagetrac-ylchapuy
Copy link
Mannequin Author

sagetrac-ylchapuy mannequin commented Jun 28, 2010

Author: Yann Laigle-Chapuy

@malb
Copy link
Member

malb commented Jun 29, 2010

Reviewer: Martin Albrecht

@malb
Copy link
Member

malb commented Jun 29, 2010

comment:2

Looks good, doctests pass.

@qed777
Copy link
Mannequin

qed777 mannequin commented Jul 20, 2010

Merged: sage-4.5.2.alpha0

@qed777 qed777 mannequin removed the s: positive review label Jul 20, 2010
@qed777 qed777 mannequin closed this as completed Jul 20, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-mvngu sagetrac-mvngu

Labels
c: cryptography p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.5.2
Development

No branches or pull requests
1 participant
@malb