Tomcat FIPS mode questions

[Christopher Schultz <ch...@christopherschultz.net>]

Chris,

I've been reviewing your patches to BZ 50570
(https://issues.apache.org/bugzilla/show_bug.cgi?id=50570) and I have a
few questions.

First, why did you change the name of the init method to initializeAPR?
The method is private and should therefore be safe to re-name, but I was
wondering why you did it. Was it to match the terminateAPR method that
performs the somewhat opposite functions?

In general, patches should change the least amount of code possible, if
for no other reason than they are easier to read and evaluate. There are
a lot of changes in here which only affect the formatting of the code,
etc. and so it's tough going making sure that nothing is changing that
shouldn't be.

I'm going to be adapting your patch to make fewer changes which I'll
re-post back to BZ. If you could verify that it's functionally
equivalent to your patch if would be very helpful. (Or, maybe you could
re-post your own patch that ignores whitespace changes).

Thanks,
-chris

Re: Tomcat FIPS mode questions

[Christopher Schultz <ch...@christopherschultz.net>]

Chris,

On 6/28/2011 2:55 PM, Christopher Schultz wrote:
> I've been reviewing your patches to BZ 50570
> (https://issues.apache.org/bugzilla/show_bug.cgi?id=50570) and I have a
> few questions.

Also, why the changes to java/org/apache/tomcat/jni/Library.java?

If there's a bug in there (it doesn't look like it has anything to do
with FIPS mode), please report it and we'll handle it separately.

Thanks,
-chris