Opened 13 years ago

Closed 13 years ago

#6877 closed defect (fixed)

[with patch, positive review] Boolean function for crypto, small bugfixes and improvement

Reported by: ylchapuy Owned by: somebody
Priority: major Milestone: sage-4.1.2
Component: cryptography Keywords:
Cc: malb Merged in: Sage 4.1.2.alpha0
Authors: Yann Laigle-Chapuy Reviewers: Martin Albrecht
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by ylchapuy)

Not even in sage, but already a bug fix...

The bug comes from the different ordering for enumerating finite fields depending on the implementation (givaro or ntl in this case).

The improvements are:

  • an option to output the truth table in hexadecimal
  • the computation of the algebraic normal form

Attachments (1)

trac_6877_Boolean_function_bugfix.patch (5.8 KB) - added by ylchapuy 13 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 13 years ago by ylchapuy

  • Description modified (diff)

Changed 13 years ago by ylchapuy

comment:2 Changed 13 years ago by ylchapuy

  • Cc malb added
  • Summary changed from Boolean function for crypto, small bugfixes and improvement to [with patch, needs review] Boolean function for crypto, small bugfixes and improvement

you need to apply #6514 (both patches) first

comment:3 Changed 13 years ago by malb

  • Summary changed from [with patch, needs review] Boolean function for crypto, small bugfixes and improvement to [with patch, positive review] Boolean function for crypto, small bugfixes and improvement

Review

  • patch looks good
  • applies cleanly against 4.1.1 + #6514
  • doctests pass on sage.math

comment:4 Changed 13 years ago by mvngu

  • Authors changed from ylchapuy to Yann Laigle-Chapuy
  • Merged in set to Sage 4.1.2.alpha0
  • Resolution set to fixed
  • Reviewers set to Martin Albrecht
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.