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

Add SBox Instance: DBlock cipher #23830

Closed
pfasante opened this issue Sep 11, 2017 · 12 comments
Closed

Add SBox Instance: DBlock cipher #23830

pfasante opened this issue Sep 11, 2017 · 12 comments

Comments

@pfasante
Copy link

The list of sboxes does not contain the sbox of the "DBlock" block cipher, see https://link.springer.com/article/10.1007/s11432-014-5219-0.

Depends on #22988

CC: leo.perrin@inria.fr

Component: cryptography

Keywords: sbox

Author: Friedrich Wiemer

Branch/Commit: 6664059

Reviewer: David Roe

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

@pfasante pfasante added this to the sage-8.1 milestone Sep 11, 2017
@pfasante
Copy link
Author

Dependencies: 22988

@pfasante pfasante self-assigned this Sep 11, 2017
@pfasante
Copy link
Author

@pfasante
Copy link
Author

Last 10 new commits:

09e1c4aiteritems not py3 compatible -> items
5fd496dMerge branch 'develop' into t/22988/add_sbox_instances
c223d51Merge branch 'develop' into t/22988/add_sbox_instances
43edbb8fixed broken PRESENT reference
dfbd71dfix merge conflict
6b8e8b1the qarma block cipher actually defines three sboxes, one was missing
9b5786aMerge branch 'develop' into t/22988/add_sbox_instances
9515fe6fixes patchbot errors
31aa2dbMerge branch 'u/asante/add_sbox_instances' of git://trac.sagemath.org/sage into t/22988/add_sbox_instances
6664059added sbox instance for DBlock block cipher

@pfasante
Copy link
Author

Commit: 6664059

@pfasante
Copy link
Author

comment:5

I'm not sure, if merging the fixed branch from #22988 is correct.
Maybe waiting for the changes to show up in the main branch and then adding the Sbox instance is a better way?

@roed314
Copy link
Contributor

roed314 commented Oct 25, 2017

comment:6

For the author, you should have your actual name rather than trac username.

@roed314
Copy link
Contributor

roed314 commented Oct 25, 2017

comment:7

Other than that, this looks fine.

A more general issue with the list of sboxes: is there any good way to check whether these sboxes are correct? It would be really easy to have a typo in the definition that nobody caught.

@roed314
Copy link
Contributor

roed314 commented Oct 25, 2017

comment:8

Also, when you list dependencies, use a # symbol so that it creates a link to the other ticket.

@roed314
Copy link
Contributor

roed314 commented Oct 27, 2017

Reviewer: David Roe

@roed314
Copy link
Contributor

roed314 commented Oct 27, 2017

Changed dependencies from 22988 to #22988

@roed314
Copy link
Contributor

roed314 commented Oct 27, 2017

Changed author from asante to Friedrich Wiemer

@vbraun
Copy link
Member

vbraun commented Oct 30, 2017

Changed branch from u/asante/add_sbox_instance__dblock_cipher to 6664059

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