On Sun, Apr 15, 2018 at 10:52 AM, Martin Townsend <mtownsend1973@xxxxxxxxx> wrote: > Sorry to be a pain but looking at the other use cases for > caam_rsa_drop_leading_zeros they check len afterwards which makes more > sense to me as temp being NULL after this operation is very unlikely > :) and I suppose we are trying to catch the case where the data is all > zeroes which wouldn't be a valid signature. You are right. I have just sent a v3 that checks len after caam_rsa_drop_leading_zeros(). Thanks