Varun Wadekar wrote at Saturday, November 12, 2011 3:23 AM:
> >> + /* assign new context to device */
> >> + ctx->dd = dd;
> >> + dd->ctx = ctx;
> >> +
> >> + if (ctx->flags & FLAGS_NEW_KEY) {
> >> + /* copy the key */
> >> + memset(dd->ivkey_base, 0, AES_HW_KEY_TABLE_LENGTH_BYTES);
> >> + memcpy(dd->ivkey_base, ctx->key, ctx->keylen);
> > these really should be writes to mutually exclusive addresses.
>
> Why?
To avoid redundant work; there's little point memset()ing a region that's
going to be copied over the top of immediately afterwards.
--
nvpublic
--
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
