Browse Source

[trac951] added a clarification comment on "truncation" to prevent
accidental remove like this.

JINMEI Tatuya 14 years ago
parent
commit
54c201f100
1 changed files with 3 additions and 3 deletions
  1. 3 3
      src/lib/cryptolink/crypto_hmac.cc

+ 3 - 3
src/lib/cryptolink/crypto_hmac.cc

@@ -183,9 +183,9 @@ public:
         try {
             Botan::SecureVector<Botan::byte> our_mac = hmac_->final();
             if (len < getOutputLength()) {
-                // Currently we don't support truncated signature.  To avoid
-                // validating too short signature accidently, we enforce the
-                // standard signature size for the moment.
+                // Currently we don't support truncated signature in TSIG (see
+                // #920).  To avoid validating too short signature accidently,
+                // we enforce the standard signature size for the moment.
                 // Once we support truncation correctly, this if-clause should
                 // (and the capitalized comment above) be removed.
                 return (false);