On Mon, Sep 24, 2018 at 4:52 AM, Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> wrote: > On Wed, 19 Sep 2018 at 04:11, Kees Cook <keescook@xxxxxxxxxxxx> wrote: >> @@ -119,7 +119,7 @@ cryptoloop_transfer(struct loop_device *lo, int cmd, >> unsigned in_offs, out_offs; >> int err; >> >> - skcipher_request_set_tfm(req, tfm); >> + skcipher_request_set_sync_tfm(req, tfm); >> skcipher_request_set_callback(req, CRYPTO_TFM_REQ_MAY_SLEEP, >> NULL, NULL); >> > > Does this work? Everything is a direct wrapper for existing types and functions, so I wouldn't expect any functional change. I haven't been able to test this particular interface, though. cryptoloop is very deprecated, isn't it? -Kees -- Kees Cook Pixel Security
