inliniac/suricata — pull request #373 by poona (2013-05-18 12:30:57+00:00)
Title: More lock fixes for the transaction update.  Issues reported by Coverity...

Some changes.

We reset the state only in ContinueDetection() only, and that is if the tx is complete. Any tx added to de_state will have to go through ContinueDetection() in the next cycle if more alstate is available, and if the tx is complete, then we carry out Continue and then reset de_state. So this way we know the de_state will no longer have stored state for the completed tx post ContinueDetection when the inspect_id is incremented further up.

Inside UpdateTransactionId we do nothing other updating the transaction id.