Re: [Perl-openssl-users] Crypt::OpenSSL::RSA 0.23 segfaults in rsa_crypt - signed>unsigned conversion

[Mark M. <Mar...@ij...>]

On Sunday November 12 2006 21:48, Ian Robertson wrote:
> Thanks for the bug report.  I've checked in a change to make to_length
> an int, instead of STRLEN.  However, before cutting a release, I'd like
> if possible to understand the core problem.  Would you mind editing your
> copy of RSA.xs to change to_length to an int, and then see what error is
> getting reported by the call to CHECK_OPEN_SSL(0)?

Thanks Ian, this now reveals the condition that lead to the failure:

$ Mail-DKIM-0.20/t/verifier.t
  ...
  # verifying message 'multiple_1.txt'
  DEBUG-OUTPUT: RSA.xs:201: OpenSSL error: data too small for key size
      at /usr/local/lib/perl5/site_perl/5.8.8/Mail/DKIM/PublicKey.pm line 305.
  #   result: pass

It is an application error, in this case probably intentional,
as it is part of installation tests for module Mail::DKIM, trying all sorts
of valid an invalid keys. It certainly had no right to crash the process. 
Thanks for looking into it!

A note to Jason Long, the author of Mail::DKIM :  the DEBUG-OUTPUT line above
was produced by a print I inserted into Mail::DKIM::Verifier::finish_body,
after desperately trying to obtain the error report. It turned out that
the "local $@" within "try" in sub finish_body should not be there, as
it prevents the error report in $@ to reach outer scope and the $E in
"otherwise" block receives a useless error string.

Regards
   Mark