#551 Patch fixing a Nullp in dep-triggered war-redeployment

JBossWeb (14)
Marian Kelc

I have a Web-Application with dependencies to a
Datasource and want the WebApp
to be redeployed when the *-ds.xml file is changed.

So I injected the Dependencies in the jboss-web.xml:






When i change the ds.xml-File Jboss tries to redeploy
the war-file, but
finishes with an exception (stacktrace is appended at
the end):

So i got to have a look to org/jboss/web/WebModule.java
and found the lines 63-71

[code]63: public synchronized void startModule()
64: throws DeploymentException
65: {
66: // Get the war URL
67: URL warURL = di.localUrl != null ? di.localUrl
: di.url;
68: WebApplication webApp = deployer.start(di);
69: di.context.put(AbstractWebContainer.WEB_APP,
70: container.addDeployedApp(warURL, webApp);
71: }[/code]

this.di becomes null on destroy in the following piece
of code (line 54-59)

[code]54: protected void destroyService()
55: {
56: this.di = null;
57: this.container = null;
58: this.deployer = null;
59: }[/code]

So i commented out the lines 56-58

[code]54: protected void destroyService()
55: {
56:// this.di = null;
57:// this.container = null;
58:// this.deployer = null;
59: }[/code]

and recompile JBoss 4.0.3 RC1. And now, it works like

And now my question:

1. Is my solution ok? Must these members be nulled on
destroy? Or can the
while destroyService-Method be removed?
2. When ok, can my Patch (ID as Sourceforge) be merged?

Thanx in advance!


Marian Kelc


  • Marian Kelc

    Marian Kelc - 2005-08-15

    Patch against server\src\main\org\jboss\web\WebModule.java

  • Adrian Brock

    Adrian Brock - 2005-08-15

    Logged In: YES

    All issues have been moved to http://jira.jboss.com. Existing
    issues have been moved. New issues will be closed with this
    canned reponse.

  • Adrian Brock

    Adrian Brock - 2005-08-15
    • status: open --> closed-postponed

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

No, thanks