Hi,

Good catch, both of them.
Fixed in revision 23251.

Thanx,
Jaap

packet steve wrote:
> 
> 
>  
> Potential infinite loop at line 608 of packet-gdsdb.c 
>  
> while(tvb_bytes_exist(tvb, 0, 12)) {
>  proto_tree_add_item(tree, hf_gdsdb_receive_direction, tvb, offset, 4, 
> FALSE);
>  offset += 4;
>  proto_tree_add_item(tree, hf_gdsdb_receive_offset, tvb, offset, 8, FALSE);
>  offset += 8;
> }
>  
> Perhaps the intention was:
>  tvb_bytes_exist(tvb, offset, 12)
> 
> 

_______________________________________________
Wireshark-dev mailing list
[email protected]
http://www.wireshark.org/mailman/listinfo/wireshark-dev

Reply via email to