mod_jserv/2855: illegal access to private members from inner classes of JServConnection

[Ari Halberstadt <ar...@shore.net>]

>Number:         2855
>Category:       mod_jserv
>Synopsis:       illegal access to private members from inner classes of JServConnection
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    jserv
>State:          open
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Fri Aug 14 11:20:00 PDT 1998
>Last-Modified:
>Originator:     ari@shore.net
>Organization:
apache
>Release:        1.3.1
>Environment:
NT4SP3, microsoft java sdk 3.0
>Description:
The inner classes JServInputStream and JServOutputStream access the private members "in" and "out", respectively, of JServConnection. This is caught by Microsoft's jview in java sdk 3.0.

This bug manifests itself as not getting any output returned from the servlet and getting an illegal access exception.
>How-To-Repeat:
Run jserv on NT with Microsoft's Java SDK v3.0 (see www.microsoft.com/java for download, I recommend using the separate .exe downloads and not trying to use their brain-damaged internet install wizard)
>Fix:
my fix was to add private "in" and "out" members to JServInputStream and JServOutputStream, respectively, and to initialize these in the constructor, e.g.,

   public JServInputStream(InputStream in) {
      available = getContentLength();
      this.in = in;
   }

a similar change is made for JServOutputStream, as well as changing the calls to the constructors to provide the "in" and "out" arguments.
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <ap...@Apache.Org> in the Cc line ]
[and leave the subject line UNCHANGED.  This is not done]
[automatically because of the potential for mail loops. ]
[If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request ]
[from a developer.                                      ]
[Reply only with text; DO NOT SEND ATTACHMENTS!         ]