Re: [Wireshark-dev] Setting column text using a Lua plugin

2013-02-12 Thread Graham Bloice
On 12 February 2013 01:52, Guy Harris g...@alum.mit.edu wrote: On Feb 11, 2013, at 5:02 PM, Hadriel Kaplan hkap...@acmepacket.com wrote: I guess I'll just fix it and have it say works for tshark only. ...but there's not even a guarantee that it'll work for TShark forever, so I changed

Re: [Wireshark-dev] Setting column text using a Lua plugin

2013-02-12 Thread Guy Harris
On Feb 12, 2013, at 2:54 AM, Graham Bloice graham.blo...@trihedral.com wrote: On 12 February 2013 01:52, Guy Harris g...@alum.mit.edu wrote: On Feb 11, 2013, at 5:02 PM, Hadriel Kaplan hkap...@acmepacket.com wrote: I guess I'll just fix it and have it say works for tshark only. ...but

[Wireshark-dev] Setting column text using a Lua plugin

2013-02-11 Thread Hadriel Kaplan
Howdy, Setting column text via Lua plugins by using the pinfo:set() method (the Pinfo from a Listener tap), doesn't seem to work for two reasons: 1) The TL_REQUIRES_COLUMNS flag wasn't being set when registering Lua listener taps in Listener_new(). I fixed that in my local copy. 2) Wireshark

Re: [Wireshark-dev] Setting column text using a Lua plugin

2013-02-11 Thread Guy Harris
On Feb 11, 2013, at 4:10 PM, Hadriel Kaplan hkap...@acmepacket.com wrote: Setting column text via Lua plugins by using the pinfo:set() method (the Pinfo from a Listener tap), doesn't seem to work for two reasons: 1) The TL_REQUIRES_COLUMNS flag wasn't being set when registering Lua

Re: [Wireshark-dev] Setting column text using a Lua plugin

2013-02-11 Thread Hadriel Kaplan
On Feb 11, 2013, at 7:37 PM, Guy Harris g...@alum.mit.edu wrote: *If* Wireshark is supposed to support setting columns in taps, rather than in dissectors or in post-dissectors: http://wiki.wireshark.org/Lua/Examples/PostDissector *then* the listener taps would have to be called

Re: [Wireshark-dev] Setting column text using a Lua plugin

2013-02-11 Thread Guy Harris
On Feb 11, 2013, at 5:02 PM, Hadriel Kaplan hkap...@acmepacket.com wrote: I guess I'll just fix it and have it say works for tshark only. ...but there's not even a guarantee that it'll work for TShark forever, so I changed your update to use only the post-dissector code.