diff options
author | Marek Vasut <[email protected]> | 2014-03-14 02:37:04 +0100 |
---|---|---|
committer | Herbert Xu <[email protected]> | 2014-03-21 21:54:20 +0800 |
commit | ab6bf4e5e5e4298e8649e635bee25542cccbfd97 (patch) | |
tree | b28d2747b827c6ca66ab3445e737a5eaf3bb6209 /lib/test-string_helpers.c | |
parent | 85e0da925b2ddbf39a6cefb4e02ad51d0a0912c0 (diff) |
crypto: hash - Fix the pointer voodoo in unaligned ahash
Add documentation for the pointer voodoo that is happening in crypto/ahash.c
in ahash_op_unaligned(). This code is quite confusing, so add a beefy chunk
of documentation.
Moreover, make sure the mangled request is completely restored after finishing
this unaligned operation. This means restoring all of .result, .base.data
and .base.complete .
Also, remove the crypto_completion_t complete = ... line present in the
ahash_op_unaligned_done() function. This type actually declares a function
pointer, which is very confusing.
Finally, yet very important nonetheless, make sure the req->priv is free()'d
only after the original request is restored in ahash_op_unaligned_done().
The req->priv data must not be free()'d before that in ahash_op_unaligned_finish(),
since we would be accessing previously free()'d data in ahash_op_unaligned_done()
and cause corruption.
Signed-off-by: Marek Vasut <[email protected]>
Cc: David S. Miller <[email protected]>
Cc: Fabio Estevam <[email protected]>
Cc: Herbert Xu <[email protected]>
Cc: Shawn Guo <[email protected]>
Cc: Tom Lendacky <[email protected]>
Signed-off-by: Herbert Xu <[email protected]>
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions