On 05/16/2013 12:53 PM, Anoop Saldanha wrote:
> In src/detect.c:
> 
>> @@ -1622,18 +1650,9 @@ int SigMatchSignatures(ThreadVars *th_v, DetectEngineCtx *de_ctx, DetectEngineTh
>>  
>>  end:
>>      /* see if we need to increment the inspect_id and reset the de_state */
>> -    if (alstate != NULL && alproto != ALPROTO_HTTP) {
>> +    if (alstate != NULL && alproto == ALPROTO_HTTP) {
> 
> THat previous commit I was talking about had this call -
> 
> AppLayerParserAlprotoSupportsTxs(uint16_t alprot).
> 
> Which returned a call if an alproto supported txs or not. The above
> location you pointed at all had this call, before I again went back to
> hardcoded http.

Not sure I understand what you're saying here. It sounds to me that we
do need the AppLayerParserAlprotoSupportsTxs-like approach here, to
support all protocols. Correct?