re: crypto: ccp - CCP SHA crypto API support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Tom Lendacky,

The patch 0ab0a1d505ab: "crypto: ccp - CCP SHA crypto API support"
from Nov 12, 2013, leads to the following static checker warning:
	drivers/crypto/ccp/ccp-crypto-sha.c:182 ccp_do_sha_update()
	warn: should 'rctx->hash_cnt << 3' be a 64 bit type?

drivers/crypto/ccp/ccp-crypto-sha.c
   180                  sg_mark_end(sg);
   181  
   182          rctx->msg_bits += (rctx->hash_cnt << 3);        /* Total in bits */
                                   ^^^^^^^^^^^^^^^^^^^
This operation wraps if the msg is over 500MB.  I'm not sure if that's
possible, but ->msg_bits is declared as a u64 here and in
rctx->cmd.u.sha.msg_bits as well.

   183  
   184          memset(&rctx->cmd, 0, sizeof(rctx->cmd));


regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux