tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Romain Manni-Bucau <rmannibu...@gmail.com>
Subject Re: leak if jspServlet.destroy() fails
Date Fri, 01 Jul 2016 16:10:46 GMT
+1


Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> |  Blog
<https://blog-rmannibucau.rhcloud.com> | Old Wordpress Blog
<http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
<http://www.tomitribe.com> | JavaEE Factory
<https://javaeefactory-rmannibucau.rhcloud.com>

2016-07-01 18:06 GMT+02:00 Mark Thomas <markt@apache.org>:

> On 01/07/2016 16:41, Romain Manni-Bucau wrote:
> > Hello guys,
> >
> > if a jspDestroy() throws an Exception then the instance manager is not
> > called which can lead to some leaks depending the implementation (see
> > org.apache.jasper.servlet.JspServletWrapper#destroy)
> >
> > Would it be possible to have a way to release/cleanup the jsp context
> > through the instance manager or to call destroyInstance properly?
>
> I think wrapping the call to destroy in a try/catch would be the way to
> go. It would be worth a check to see if there are any other places where
> similar issues could occur with JSPs.
>
> Mark
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message