[jboss-jira] [JBoss JIRA] Updated: (JBRULES-1776) Lock on AbstractRuleBase object is not released when an error occurs.

[Edson Tirelli (JIRA)]

     [ https://jira.jboss.org/jira/browse/JBRULES-1776?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Edson Tirelli updated JBRULES-1776:
-----------------------------------

    Fix Version/s: 5.0.0.CR1
                       (was: 5.0.0.M3)


> Lock on AbstractRuleBase object is not released when an error occurs.
> ---------------------------------------------------------------------
>
>                 Key: JBRULES-1776
>                 URL: https://jira.jboss.org/jira/browse/JBRULES-1776
>             Project: JBoss Drools
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 4.0.7
>         Environment: Windows XP, jdk1.5.0_01
>            Reporter: Le Huynh
>            Assignee: Edson Tirelli
>             Fix For: 5.0.0.CR1
>
>
> When modifying the rules using addPackage() or removeRule() from the AbstractRulebase class this would lock() the object.  But if any error occurs after the lock() is called, this object's lock is never released as it would never reach the line to call unlock().  It appears that the removePackage() in the AbstractRulebase is the only method that has a try{} finally{} to unlock if there is an error.
> Below is stack of removing a rule, when this occurs, all other threads will wait forever when trying to modify rules.
> "TP-Processor20" daemon prio=10 tid=0x83adcc00 nid=0x4af5 in Object.wait() [0x82231000..0x82232f30]
>    java.lang.Thread.State: WAITING (on object monitor)
> 	at java.lang.Object.wait(Native Method)
> 	- waiting on <0x964babb0> (a org.drools.util.concurrent.locks.ReentrantLock$NonfairSync)
> 	at java.lang.Object.wait(Object.java:485)
> 	at org.drools.util.concurrent.locks.ReentrantLock$NonfairSync.lock(ReentrantLock.java:106)
> 	- locked <0x964babb0> (a org.drools.util.concurrent.locks.ReentrantLock$NonfairSync)
> 	at org.drools.util.concurrent.locks.ReentrantLock.lock(ReentrantLock.java:195)
> 	at org.drools.common.AbstractRuleBase.lock(AbstractRuleBase.java:343)
> 	at org.drools.common.AbstractRuleBase.removeRule(AbstractRuleBase.java:630)
> 	- locked <0x964aa710> (a java.util.HashMap)
> 	at com.prototype.rules.RulesEngine.removeRuleFromEngine(RulesEngine.java:207)
> 	at com.prototype.rules.RulesEngine.updateRule(RulesEngine.java:137)
> 	- locked <0x964a6208> (a com.prototype.rules.RulesEngine)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira