I for one fully appreciate this change, and would like to see it in the code the soonest possible. (IMHO, it’s been long overdue.)
On Sep 30, 2015, at 16:21 , Jeffrey Walton <[email protected]> wrote: > Hi Everyone, > > We need to change ECIES in the near future. For 5.6.3, I want to add the > deprecated attribute to warn folks about the upcoming breaking change. > > Here's the message when compiling: > > c++ -DNDEBUG -g3 -O2 -Wall -fPIC -march=native -DCRYPTOPP_DISABLE_ASM -pipe > -c bench2.cpp > bench2.cpp:293:3: warning: 'ECIES' is deprecated: ECIES will be changing in > the > near future due to (1) an implementation bug and (2) an interop issue. > [-Wdeprecated-declarations] > ECIES<ECP>::Encryptor cpub(cpriv); > ^ > > The message shows when an ECIES class is instantiated; and not by including > the eccrypto.h header. > > The implementation bug is: Crypto++ writes the length of the label in bytes, > not bits. The interop bug is with Bouncy Castle: the label needs to be 8 > octets and not 4 octets. > > Does anyone have objections to the attribute, its verbiage/text, or other > thoughts? > > Jeff -- -- You received this message because you are subscribed to the "Crypto++ Users" Google Group. To unsubscribe, send an email to [email protected]. More information about Crypto++ and this group is available at http://www.cryptopp.com. --- You received this message because you are subscribed to the Google Groups "Crypto++ Users" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
